D5515: Update the ungrabMouse hack for Qt 5.8

2017-04-24 Thread Anthony Fieroni
anthonyfieroni added a comment. David, it's sould be done in plasma-framework too. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D5515 To: davidedmundson, #plasma, mart Cc: anthonyfieroni, mvourlakos, plasma-devel, #frameworks, spstarr, progwolff,

D5515: Update the ungrabMouse hack for Qt 5.8

2017-04-24 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R242:29f1ee385891: Update the ungrabMouse hack for Qt 5.8 (authored by davidedmundson). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5515?vs=13747&i

D5515: Update the ungrabMouse hack for Qt 5.8

2017-04-24 Thread David Edmundson
davidedmundson updated this revision to Diff 13747. davidedmundson added a comment. Rewrote with a runtime check and a link to a qt bug report REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5515?vs=13609&id=13747 BRANCH origin-master (

D5515: Update the ungrabMouse hack for Qt 5.8

2017-04-20 Thread Marco Martin
mart added a comment. do you know off hand a list of places where this is needed? i think kicker,taskmanager,systray... anything else? REPOSITORY R242 Plasma Framework (Library) BRANCH origin-master (branched from master) REVISION DETAIL https://phabricator.kde.org/D5515 To: davided

D5515: Update the ungrabMouse hack for Qt 5.8

2017-04-20 Thread Marco Martin
mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) BRANCH origin-master (branched from master) REVISION DETAIL https://phabricator.kde.org/D5515 To: davidedmundson, #plasma, mart Cc: anthonyfieroni, mvourlakos, plasma-d

D5515: Update the ungrabMouse hack for Qt 5.8

2017-04-19 Thread Anthony Fieroni
anthonyfieroni added a comment. https://phabricator.kde.org/D4712 https://phabricator.kde.org/D4711 It has one unfixible case, dismiss context menu by clicking on onther window. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D5515 To: davidedmu

D5515: Update the ungrabMouse hack for Qt 5.8

2017-04-19 Thread Michail Vourlakos
mvourlakos added a comment. I had reported qts faulty behavior concerning centextmenus at: https://bugreports.qt.io/browse/QTBUG-59044 I think it is the mentioned case... REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D5515 To: davidedmundson

D5515: Update the ungrabMouse hack for Qt 5.8

2017-04-19 Thread David Edmundson
davidedmundson created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY There was an unidentified bug where when opening a context menu in response to a mousePress event led to the nex