Cancel code completion and code hint upon go-to line (bookmark, breakpoint, etc.)

[jaydensipe]

Fixes #95056.

When doing an action that does a "go-to line" such as Go to Next/Previous Breakpoint or Go to Next/Previous Bookmark, this ensures the code hint or currently open code completion is cleared when jumping lines.

[HolonProduction]

Can confirm that this fixes the linked issue. Code looks straight forward.

[kitbdev]

This works for the linked issue, but there are more places where the code hint and code completion need to be hidden.
For both: using the Search bar.
For the code hint: ctrl+up and down when the caret moves, left and right that goes out of the function, selecting text and using ctrl+d to select next, ctrl+x to cut the existing line, etc.
Also they shouldn't be hidden when just scrolling (using scrollbar works, but mouse wheel hides it).

Ideally CodeEdit should respond to all caret movements and cancel it when needed there instead of needing to cancel it in each of these separately. This can be done in another PR if we want to merge this one as is.

[akien-mga]

I thought I was merging this but it seems I missed it in my batch. Good timing as kleonc had some comments :D

[HolonProduction]

Ideally CodeEdit should respond to all caret movements and cancel it when needed there instead of needing to cancel it in each of these separately. This can be done in another PR if we want to merge this one as is.

I'd say adjusting the search bar behavior here would make sense.

A more general solution on CodeEdit level sounds great, but it might get complicated to generalize the decision whether the hint needs to be canceled and when it needs to be updated by querying the ScriptLanguage (e.g. when moving the cursor inside the same call we would need to update the hint, not cancel it, to highlight the correct current argument). Querying too often might also have performance implications for GDScript at the moment, due to the lack of caching. So merging this as a simple fix for now would make sense to me.

[jaydensipe]

Ideally CodeEdit should respond to all caret movements and cancel it when needed there instead of needing to cancel it in each of these separately. This can be done in another PR if we want to merge this one as is.

I'd say adjusting the search bar behavior here would make sense.

A more general solution on CodeEdit level sounds great, but it might get complicated to generalize the decision whether the hint needs to be canceled and when it needs to be updated by querying the ScriptLanguage (e.g. when moving the cursor inside the same call we would need to update the hint, not cancel it, to highlight the correct current argument). Querying too often might also have performance implications for GDScript at the moment, due to the lack of caching. So merging this as a simple fix for now would make sense to me.

Alright, updated to clear when using "Go to Line..." and when finding a successful search in the search bar.

[HolonProduction]

The search bar adjustment works as well. As said before I think merging this fix is a good idea since a generalized fix on CodeEdit level may pose some challenges at the moment.

[akien-mga]

Thanks! And congrats for your first merged Godot contribution 🎉