D4711: Ungrab mouse on menu close

2017-05-13 Thread Anthony Fieroni
anthonyfieroni abandoned this revision. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4711 To: anthonyfieroni, #plasma, mart, broulik, davidedmundson Cc: mvourlakos, plasma-devel, #frameworks, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, je

D4711: Ungrab mouse on menu close

2017-04-29 Thread Anthony Fieroni
anthonyfieroni updated this revision to Diff 14014. REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4711?vs=11600&id=14014 REVISION DETAIL https://phabricator.kde.org/D4711 AFFECTED FILES src/scriptengines/qml/plasmoid/appletinterface.cp

D4711: Ungrab mouse on menu close

2017-04-29 Thread Anthony Fieroni
anthonyfieroni added a reviewer: davidedmundson. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4711 To: anthonyfieroni, #plasma, mart, broulik, davidedmundson Cc: mvourlakos, plasma-devel, #frameworks, spstarr, progwolff, lesliezhai, ali-mohamed, je

[Differential] [Commented On] D4711: Ungrab mouse on menu close

2017-02-26 Thread Anthony Fieroni
anthonyfieroni added a comment. Yep, same faulty beharior present in all Qt apps, Qupzilla, QtCreator, etc. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4711 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: a

[Differential] [Commented On] D4711: Ungrab mouse on menu close

2017-02-24 Thread Michail Vourlakos
mvourlakos added a comment. In https://phabricator.kde.org/D4711#89334, @mart wrote: > this on released Qt 5.8? on 5.8 branch? > i can't reproduce with my 5.8 build here. > anyways, that beahvior change would be a Qt bug (one of the many the terrible 5.8 release had) and should be f

[Differential] [Commented On] D4711: Ungrab mouse on menu close

2017-02-24 Thread Marco Martin
mart added a comment. this on released Qt 5.8? on 5.8 branch? i can't reproduce with my 5.8 build here. anyways, that beahvior change would be a Qt bug (one of the many the terrible 5.8 release had) and should be fixed upstream the indicated if (!enter || !QWidget::mouseGrabber()) { code

[Differential] [Commented On] D4711: Ungrab mouse on menu close

2017-02-22 Thread Michail Vourlakos
mvourlakos added a comment. I just noticed... you dont have to go too far... any qt application using 5.8 has this problem... even qtcreator... showing a menu and then deactivating it from outside areas then the window needs a click from the user in order for this window to react

[Differential] [Commented On] D4711: Ungrab mouse on menu close

2017-02-22 Thread Michail Vourlakos
mvourlakos added a comment. You can observe the faulty behavior even with Qt's "Menu's Example". 1. Open any menu you want either with right click or from the menu 2. Close it by clicking in any outside area you want, that is (not inside the menu example window and not in the menu) 3.

[Differential] [Commented On] D4711: Ungrab mouse on menu close

2017-02-22 Thread Anthony Fieroni
anthonyfieroni added a comment. I'm not pretty sure, but https://code.woboq.org/qt5/qtbase/src/widgets/kernel/qwidgetwindow.cpp.html#374 on click at other window we have enter && mouseGrabber REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D471

[Differential] [Commented On] D4711: Ungrab mouse on menu close

2017-02-22 Thread Michail Vourlakos
mvourlakos added a comment. In https://phabricator.kde.org/D4711#88621, @mart wrote: > do you know exactly what/where changed? sorry if this is irrelevant... I dont know if the above code works but in my code tree in latte dock I can not do anything to ungrabMouse in Qt 5.8 in c

[Differential] [Updated] D4711: Ungrab mouse on menu close

2017-02-22 Thread Marco Martin
mart added a comment. do you know exactly what/where changed? REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D4711 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: anthonyfieroni, #plasma, broulik, mart Cc: plas

[Differential] [Request, 14 lines] D4711: Ungrab mouse on menu close

2017-02-21 Thread Anthony Fieroni
anthonyfieroni created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY Qt 5.8 has a bit changed and ungrabbing at show does not work. REPOSITORY R242 Plasma Framework (Library) REVI