Bug 475789 - The "Clear All" button only clears the text field but doesn't close the search view
Summary: The "Clear All" button only clears the text field but doesn't close the searc...
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: effects-overview (other bugs)
Version First Reported In: master
Platform: Compiled Sources Other
: NOR minor
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords: qt6
Depends on:
Blocks:
 
Reported: 2023-10-18 09:46 UTC by Fushan Wen
Modified: 2023-10-25 11:52 UTC (History)
2 users (show)

See Also:
Latest Commit: https://invent.kde.org/plasma/kwin/-/commit/4bdb94c87fda540cf335e0ff53652e4ea18ea19d
Version Fixed/Implemented In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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