Skip to content

Commit

Permalink
Merge pull request #1947 from uklotzde/2.1_lp1635087_clear_search_tim…
Browse files Browse the repository at this point in the history
…eout

Respect debouncing timeout when clearing the search field
  • Loading branch information
daschuer authored Dec 17, 2018
2 parents fd15f29 + fbaf526 commit d60ac3c
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions src/widget/wsearchlineedit.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -207,8 +207,11 @@ bool WSearchLineEdit::event(QEvent* pEvent) {
void WSearchLineEdit::clearSearch() {
DEBUG_ASSERT(m_state == State::Active);
setText(kEmptySearch);
// Enforce immediate update of track table
triggerSearch();
// Clearing the edit field will engage the debouncing timer
// and gives the user the chance for entering a new search
// before returning the whole (and probably huge) library.
// No need to manually trigger a search at this point!
// See also: https://bugs.launchpad.net/mixxx/+bug/1635087
}

// slot
Expand Down

0 comments on commit d60ac3c

Please sign in to comment.