Review Request 129086: Update applet alternatives menu entry visibility on demand
David Edmundson
david at davidedmundson.co.uk
Mon Oct 3 10:45:11 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129086/
-----------------------------------------------------------
(Updated Oct. 3, 2016, 10:45 a.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks and Plasma.
Repository: plasma-framework
Description
-------
It's reasnoble to expect people to download an applet just before trying
to change an existing applet using the alternatives system.
Therefore doing the check that any alternative applets exist during applet initialisation time is semantically
wrong as the option will be prematurely disabled.
This patch moves it to be verified in contextualActionsAboutToShow.
This patch also gives a startup performance boost.
Loading all the applets metadata takes a small amount of time (~50 milliseconds) not really noticable when you're doing it once, but it adds up when we're doing it for a lot of applets in a row.
Diffs
-----
src/plasma/applet.cpp 5e278dc69055de0316b49decc08e471081e34b53
src/plasma/private/applet_p.cpp 0f37fe5e3a3c417e079f54b6d8b45fa0051684a5
Diff: https://git.reviewboard.kde.org/r/129086/diff/
Testing
-------
Right click on kickoff, option was there and worked
Right click on system tray, no option
Locked widges, option wasn't there
Unlocked widgets, option in kickoff came back
Context menu still appears "instantly"
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161003/ac3760a8/attachment.html>
More information about the Kde-frameworks-devel
mailing list