Review Request 128640: Keep compatiable slot createApplet with Frameworks 5.24 *THIS NEEDS BACKPORTING TO 5.25*

Marco Martin notmart at gmail.com
Wed Aug 10 08:30:24 UTC 2016


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


Ship it!




aww, crap, didn't see that :/
thanks for noticing

- Marco Martin


On Aug. 9, 2016, 11:03 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128640/
> -----------------------------------------------------------
> 
> (Updated Aug. 9, 2016, 11:03 p.m.)
> 
> 
> Review request for KDE Frameworks, Plasma and David Faure.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> Commit bb70febdbe397b617e5c41aff5494fdbc185fa88 changed the slot for
> adding createApplet, turning the final argument from QPoint to QRect.
> 
> If the rectangle size is nothing, it acts like the original code.
> 
> Despite this being private API (ish) there is a hacks in plasma-
> workspace that call methods on the view via QMetaObject invoke. This
> obviously fails. As we need compatibility for Plasma/5.7 and earlier a
> compatibility slot needs to stay.
> 
> 
> *THIS NEEDS BACKPORTING TO 5.25*
> 
> 
> Diffs
> -----
> 
>   src/scriptengines/qml/plasmoid/containmentinterface.h c058f8358b4aa123749959a3de5b0667d7a1fecc 
>   src/scriptengines/qml/plasmoid/containmentinterface.cpp 4efc1e109bbab8ef43b686be25574ed5bd66d9ce 
> 
> Diff: https://git.reviewboard.kde.org/r/128640/diff/
> 
> 
> Testing
> -------
> 
> Before: alternatives context menu on the desktop just deleted the applet
> Now: works
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

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


More information about the Plasma-devel mailing list