Review Request 129263: Don't show "Alternatives" menu by default

David Edmundson david at davidedmundson.co.uk
Wed Oct 26 10:56:59 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129263/
-----------------------------------------------------------

Review request for Plasma.


Repository: plasma-framework


Description
-------

And only enable it in the contextMenuAboutToShow connection.

We currently enable/disable the menu in this slot, but we default to it on. This meant other users of actions() that can't correctly emit contextMenuAboutToShow gain an extra entry which might not be relevant.


Diffs
-----

  src/plasma/private/applet_p.cpp bfb6d2502a2fc804497e571e7000e20a40a20d5c 

Diff: https://git.reviewboard.kde.org/r/129263/diff/


Testing
-------

Appears on kickoff
Doesn't appear on the system tray

(as before)

No longer appears in the toolbox (new fix)


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161026/f5ec4fec/attachment-0001.html>


More information about the Plasma-devel mailing list