Bug 475789

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-overviewAssignee: 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: Version Fixed/Implemented In:
Sentry Crash Report:

Description Fushan Wen 2023-10-18 09:46:28 UTC
kwin at d91534303e319316772f5039a7938c994ae8428b
plasma-framework at ca13de437f4d182e66d31d09b4f67f6657e775df
Qt 6.6.0
X11
Comment 1 Nate Graham 2023-10-20 15:25:12 UTC
Niccolò, can you take a look?
Comment 2 Bug Janitor Service 2023-10-24 20:35:55 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/4569
Comment 3 David Edmundson 2023-10-25 11:52:59 UTC
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