Bug 445707 - When Milou view is visible, clicking on empty space where a window used to be activates that window instead of just closing the effect
Summary: When Milou view is visible, clicking on empty space where a window used to be...
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: effects-overview (other bugs)
Version First Reported In: git master
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2021-11-18 17:03 UTC by Nate Graham
Modified: 2021-11-19 15:27 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments

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