Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add searching by event in Editor Settings shortcuts and Project Settings input map. #66770

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

EricEzaM
Copy link
Contributor

@EricEzaM EricEzaM commented Oct 2, 2022

  • Focus into the LineEdit, then perform input to search the list of events by the events assigned.
  • New specialised editor-only control for this: EventListenerLineEdit. Line edit is a good candidate for such a control because you can focus it, override it's input handling, and show the event all in one control.

Update InputEventConfigurationDialog to use event listener line edit rather than the separate tabs.

  • Cleaner look - no need for tabs.
  • Simpler code.

Please note the preview text in the LineEdit has now been changed from what is in the videos below. New is like so Unfocused: "Filter by event..." Focused: "Listening for input..."

Shortcuts Search

godot.windows.editor.dev.x86_64_WS0WTY1oqO.mp4

Input map Search

godot.windows.editor.dev.x86_64_WkMVTLmtPJ.mp4

New Event Picker (no more tabs!) (can now pick escape from the keyboard and close the window with escape too!)

godot.windows.editor.dev.x86_64_HQ7AsoJMa3.mp4

@@ -35,6 +35,100 @@
#include "editor/editor_scale.h"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be time to split this file up. It contains 3 classes now. I would do it in another pr tho.

@EricEzaM EricEzaM force-pushed the search-by-event branch 2 times, most recently from 0af967a to f4dbd96 Compare October 2, 2022 16:18
@Calinou Calinou added this to the 4.0 milestone Oct 2, 2022
@EricEzaM EricEzaM force-pushed the search-by-event branch 2 times, most recently from 9636422 to dacfd1e Compare October 3, 2022 03:56
Copy link
Member

@bruvzg bruvzg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like a good feature to have:

A minor issue - it's possible to input text input the filter LineEdit if you keep pressing the same letter key, e.g., pressing F multiple type will result in fffffffF in the filter.

Also, I'm not sure about "Filter by event... (perform input here)", it's a bit too long. Maybe it's better to display only Filter by event... by default and change it to Listening for input... when it's focused, or something like this.

@EricEzaM
Copy link
Contributor Author

EricEzaM commented Oct 3, 2022

Also, I'm not sure about "Filter by event... (perform input here)", it's a bit too long. Maybe it's better to display only Filter by event... by default and change it to Listening for input... when it's focused, or something like this.

What a great idea, can't believe I didn't think of this!

…ings input map.

* Focus into the LineEdit, then perform input to search the list of events by the events assigned.
* New specialised editor-only control for this: EventListenerLineEdit. Line edit is a good candidate for such a control because you can focus it, override it's input handling, and show the event all in one control.
Update InputEventConfigurationDialog to use event listener line edit rather than the separate tabs.
* Cleaner look - no need for tabs.
* Simpler code.
@EricEzaM
Copy link
Contributor Author

EricEzaM commented Oct 3, 2022

Those 2 are fixed now, thanks!

@akien-mga akien-mga merged commit 132a0e3 into godotengine:master Oct 5, 2022
@akien-mga
Copy link
Member

Thanks!

@kristin-meshka
Copy link

Don't know if this is really the right place too put this, but i found a bug with the input search. That when you press Esc it will close the window while listening for inputs.

inputsDontCanelOutEsc.mp4

@KoBeWi
Copy link
Member

KoBeWi commented Dec 20, 2022

Don't know if this is really the right place too put this

It's not, open a new issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants