[Differential] [Commented On] D3085: Use DBusMenu if available

2017-01-10 Thread Kai Uwe Broulik
broulik added a comment. You figured that stuf out now, right? REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D3085 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: davidedmundson, broulik,

[Differential] [Commented On] D3085: Use DBusMenu if available

2017-01-08 Thread David Edmundson
davidedmundson added a comment. > Dunno about the applet. The applet requires it. Dynamic bookmarks just don't work otherwise. REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D3085 EMAIL PREFERENCES

[Differential] [Commented On] D3085: Use DBusMenu if available

2017-01-07 Thread broulik (Kai Uwe Broulik)
broulik added a comment. Meh. For the title bar button shouldn't matter, though, as there I create a new importer everytime you click the button, so the menu is always up-to-date. Dunno about the applet. REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL

[Differential] [Commented On] D3085: Use DBusMenu if available

2017-01-06 Thread davidedmundson (David Edmundson)
davidedmundson added a comment. One teeny tiny catastrophe remains. In the spec calling AboutToShow returns whether the menu changed or not and if you need to refresh it. Our client code relies on this happening This QPT always returns false. The relevant code is in the QPT, but