Bug 445707

Summary: When Milou view is visible, clicking on empty space where a window used to be activates that window instead of just closing the effect
Product: [Plasma] kwin Reporter: Nate Graham <nate>
Component: effects-overviewAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal Keywords: regression
Priority: NOR    
Version First Reported In: git master   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Nate Graham 2021-11-18 17:03:47 UTC
Git master everything


STEPS TO REPRODUCE
1. Open Overview effect
2. Position cursor above a window that is off to the right or left side (where the Milou view will not appear)
3. Type something in the search field to show the Milou view and hide all windows
4. Click


OBSERVED RESULT
The effect closes and the window that you were hovering over before is activated, even though it was invisible at the moment when you clicked.


EXPECTED RESULT
The effect closes without explicitly activating any windows.
Comment 1 Bug Janitor Service 2021-11-19 07:43:31 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/1690
Comment 2 Vlad Zahorodnii 2021-11-19 15:27:44 UTC
Git commit 17dbdf89bdcbb5788e8b0f9dcab83302b8912c4b by Vlad Zahorodnii.
Committed on 19/11/2021 at 15:27.
Pushed by vladz into branch 'master'.

effects/overview: Prevent WindowHeap accepting mouse events while searching

If the opacity property is set to 0, the item will still receive mouse
events. One either needs to set the enabled or the visible property to
false to ensure that the WindowHeap receives no mouse events.

The reason for setting the opacity to 0 instead of the visible property
to false was that I thought later will nuke the paint node, but it seems
like that's not the case.

M  +2    -20   src/effects/overview/qml/ScreenView.qml

https://invent.kde.org/plasma/kwin/commit/17dbdf89bdcbb5788e8b0f9dcab83302b8912c4b