You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
any project that uses signals and connects them via the editor
Describe the problem or limitation you are having in your project
When using the Node dock to connect a signal, you are presented with two windows, the first Connect a Signal to a Method and a second Select Method. In the Select Method window, it has a search box, however this search is case sensitive. While snake_case is the convention, it is not enforced meaning scripts that have camel or pascal case methods will find this search more difficult to use
on search
Describe the feature / enhancement and how it helps to overcome the problem or limitation
If this search was done in a case insensitive way it will be easier for users to find methods when connecting signals in the editor
Describe how your proposal will work, with code, pseudo-code, mock-ups, and/or diagrams
I'm unsure how the other search boxes in engine handle, but in local testing I was able to achieve this by changing
If this enhancement will not be used often, can it be worked around with a few lines of script?
I don't believe this can be fixed with a script/addon
Is there a reason why this should be core and not an add-on in the asset library?
This is core editor functionality. It also appears other parts of the editor already have case insensitive searches (for example Connect a Signal to a Method) (compare below screenshot to one above)
The text was updated successfully, but these errors were encountered:
Describe the project you are working on
any project that uses signals and connects them via the editor
Describe the problem or limitation you are having in your project
When using the Node dock to connect a signal, you are presented with two windows, the first
Connect a Signal to a Method
and a secondSelect Method
. In theSelect Method
window, it has a search box, however this search is case sensitive. While snake_case is the convention, it is not enforced meaning scripts that have camel or pascal case methods will find this search more difficult to useon search
Describe the feature / enhancement and how it helps to overcome the problem or limitation
If this search was done in a case insensitive way it will be easier for users to find methods when connecting signals in the editor
Describe how your proposal will work, with code, pseudo-code, mock-ups, and/or diagrams
I'm unsure how the other search boxes in engine handle, but in local testing I was able to achieve this by changing
https://github.com/godotengine/godot/blob/7ebc866418b075df58cbe4e31fcf8b0c3acd70a1/editor/connections_dialog.cpp#L285
if (!p_search_string.is_empty() && !mi.name.contains(p_search_string)) {
to
if (!p_search_string.is_empty() && !mi.name.to_lower().contains(p_search_string.to_lower())) {
and it appeared to fix this. I have submitted a PR godotengine/godot#91598
If this enhancement will not be used often, can it be worked around with a few lines of script?
I don't believe this can be fixed with a script/addon
Is there a reason why this should be core and not an add-on in the asset library?
This is core editor functionality. It also appears other parts of the editor already have case insensitive searches (for example
Connect a Signal to a Method
) (compare below screenshot to one above)The text was updated successfully, but these errors were encountered: