https://bugs.kde.org/show_bug.cgi?id=377838

David Edmundson <k...@davidedmundson.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/pla
                   |                            |sma-workspace/1cb0d180bdda8
                   |                            |bf06b77daf7a404d9213270752b
             Status|CONFIRMED                   |RESOLVED

--- Comment #2 from David Edmundson <k...@davidedmundson.co.uk> ---
Git commit 1cb0d180bdda8bf06b77daf7a404d9213270752b by David Edmundson.
Committed on 28/01/2018 at 09:14.
Pushed by davidedmundson into branch 'Plasma/5.12'.

Fix autohide panels on wayland

Summary:
To test if the panel is under the cursor we used:
if (geometry().contains(QCursor::pos(screenToFollow()))) {

Unsurprisingly in wayland we don't know the cursor position once it's
left our window. Behaviour seems to be undefined.

We were already using enter and leave events to start the autohide
timer, so we may as well rely on that for tracking state too.

Test Plan:
Tested mouse in, mouse out

Tested mouse in and waiting and panel stayed open

Tested opening wifi applet and closing it
That will trigger the restoreAutoHide method as an applet status changed
The panel stayed open because the mouse was on it.

Reviewers: #plasma, mart

Reviewed By: #plasma, mart

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D10101

M  +20   -18   shell/panelview.cpp
M  +1    -0    shell/panelview.h

https://commits.kde.org/plasma-workspace/1cb0d180bdda8bf06b77daf7a404d9213270752b

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to