Review Request: Containments expose an AppletContainer

Marco Martin notmart at gmail.com
Wed Oct 24 19:41:35 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106817/#review20818
-----------------------------------------------------------

Ship it!


i think this is a good approach.

only thing it can work only until we have only c++ stuff, so toolboxes or applet handles would have to be in a traditional import


plasma/scriptengines/javascript/plasmoid/appletcontainer.h
<http://git.reviewboard.kde.org/r/106817/#comment16438>

    those files should perhaps be in the "declarative" subdirectory in the script engine


- Marco Martin


On Oct. 13, 2012, 12:43 a.m., Sebastian Kügler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106817/
> -----------------------------------------------------------
> 
> (Updated Oct. 13, 2012, 12:43 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
>     This adds a class to manipulate applets in more detail in context of
>     containments. The type will only be available from Containments, not
>     Applets of PopupApplets.
>     
>     This binds the necessary functionality of Applets to build fully
>     features containments. The code has been moved from MobileComponents,
>     untangling another piece there and putting the functionality where it
>     belongs, into the specific component type bindings to allow this functionality 
>     also outside of plasma-mobile and further reduce the delta between the shell 
>     implementations.
> 
>     Intended for kde-runtime master.
> 
> 
> Diffs
> -----
> 
>   plasma/scriptengines/javascript/CMakeLists.txt d259b7f 
>   plasma/scriptengines/javascript/plasmoid/appletcontainer.h PRE-CREATION 
>   plasma/scriptengines/javascript/plasmoid/appletcontainer.cpp PRE-CREATION 
>   plasma/scriptengines/javascript/plasmoid/appletinterface.cpp 46a644a 
> 
> Diff: http://git.reviewboard.kde.org/r/106817/diff/
> 
> 
> Testing
> -------
> 
> The code has been used for some time in Plasma Active, so can be considered stable and well tested. Also tested with a new Containment, works as expected.
> 
> 
> Thanks,
> 
> Sebastian Kügler
> 
>

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


More information about the Plasma-devel mailing list