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

Automatically start searching in the asset library when entering text #42402

Merged
merged 1 commit into from
Sep 29, 2020

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Sep 29, 2020

Entering text will now start searching automatically after 0.25 seconds have passed (debounce delay).

This removes the need for a separate Search button. For reference, this "automatically searching" search bar design is also found in Visual Studio Code and JetBrains IDEs.

@Calinou Calinou added cherrypick:3.x Considered for cherry-picking into a future 3.x release enhancement topic:assetlib topic:editor usability labels Sep 29, 2020
@Calinou Calinou added this to the 4.0 milestone Sep 29, 2020
Entering text will now start searching automatically after 0.25 seconds
have passed (debounce delay).

This removes the need for a separate Search button.
@Calinou Calinou changed the title Automatically start searching in the asset library when entering textr Automatically start searching in the asset library when entering text Sep 29, 2020
@Calinou Calinou force-pushed the assetlib-search-auto-debounce branch from 60d2450 to 6055db2 Compare September 29, 2020 10:46
@akien-mga akien-mga merged commit 74da12b into godotengine:master Sep 29, 2020
@akien-mga
Copy link
Member

Thanks!

@akien-mga
Copy link
Member

Cherry-picked for 3.2.4.

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Sep 29, 2020
@Calinou Calinou deleted the assetlib-search-auto-debounce branch September 30, 2020 20:46
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.

2 participants