Review Request 127217: Move contextualActionsAboutToShow to Applet

Marco Martin notmart at gmail.com
Mon Feb 29 12:05:08 UTC 2016


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


Ship it!




Ship It!

- Marco Martin


On Feb. 29, 2016, noon, David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127217/
> -----------------------------------------------------------
> 
> (Updated Feb. 29, 2016, noon)
> 
> 
> Review request for KDE Frameworks and Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> Applet contains the menu actions, therefore notifying the menu needs updating should
> happen in Applet not AppletInterface. With AppletInterface being a dumb
> wrapper.
> 
> This exposes functionality to anyone who overrides Applet, but also will
> also allows system tray to emit this signal without having to use
> private API.
> 
> This is a better version of my patch at https://git.reviewboard.kde.org/r/126811/ to fix 358119 that I wasn't happy with.
> 
> 
> Diffs
> -----
> 
>   src/plasma/applet.h df57e945b7dcd3b907c68ae82e5a26f2caeb5c9f 
>   src/scriptengines/qml/plasmoid/appletinterface.cpp 550438d05640cc02a1a2bda546601e6740a9b4e9 
>   src/scriptengines/qml/plasmoid/containmentinterface.cpp 44ed56ed85adec4af054a25c1cc5b46605152320 
> 
> Diff: https://git.reviewboard.kde.org/r/127217/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160229/a060db75/attachment.html>


More information about the Kde-frameworks-devel mailing list