bshah created this revision. bshah added reviewers: Plasma, mart, graesslin. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY If we don't pass the current screen to QCursor::pos, it will return position relative to primary screen, which obviously won't be in panel geometry. Instead pass it currentScreen, this way we don't filter out the mouse events for panel. BUG: 368790 FIXED-IN: 5.8.1 TEST PLAN added debug statements and verified that I get correct screen position and as well, panel is now interactive on secondary screen. REPOSITORY rPLASMAWORKSPACE Plasma Workspace BRANCH fix-panel (branched from Plasma/5.8) REVISION DETAIL https://phabricator.kde.org/D2960 AFFECTED FILES shell/panelview.cpp EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: bshah, #plasma, mart, graesslin Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
