| Summary: | The "Clear All" button only clears the text field but doesn't close the search view | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | Fushan Wen <qydwhotmail> |
| Component: | effects-overview | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | nate, niccolo.venerandi |
| Priority: | NOR | Keywords: | qt6 |
| Version First Reported In: | master | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Other | ||
| See Also: | https://bugs.kde.org/show_bug.cgi?id=475684 | ||
| Latest Commit: | https://invent.kde.org/plasma/kwin/-/commit/4bdb94c87fda540cf335e0ff53652e4ea18ea19d | Version Fixed/Implemented In: | |
| Sentry Crash Report: | |||
|
Description
Fushan Wen
2023-10-18 09:46:28 UTC
Niccolò, can you take a look? A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/4569 Git commit 4bdb94c87fda540cf335e0ff53652e4ea18ea19d by David Edmundson. Committed on 25/10/2023 at 13:52. Pushed by davidedmundson into branch 'master'. Overview: Fix clear button working in overview search onTextEdited only updates on user-invoked key presses, and not programatic changes. This is a better choice when we have a two-way bindings such as with configuration files, but in this case we always want the UI to reflect the current contents of the search box making onTextChanged the better signal. M +2 -1 src/plugins/overview/qml/main.qml https://invent.kde.org/plasma/kwin/-/commit/4bdb94c87fda540cf335e0ff53652e4ea18ea19d |