Review Request 125968: WIP: Install .plasmoid file when dropping it onto containment

[Kai Uwe Broulik]

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

(Updated Nov. 23, 2015, 5:22 nachm.)


Review request for Plasma and KDE Usability.


Changes
-------

- Use update() instead of install()
- Ignore if applet is already installed (just add it to the containment then)

I will look at the menus (cue: sections) separately


Repository: plasma-framework


Description
-------

This allows to drop a .plasmoid file on a panel or desktop and have it automatically installed and placed there.

Video: https://www.youtube.com/watch?v=5MnWnt_u5dU


Diffs (updated)
-----

  src/scriptengines/qml/CMakeLists.txt 2aedf0b 
  src/scriptengines/qml/plasmoid/containmentinterface.cpp 18be65f 

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


Testing (updated)
-------

Works

2.) How can I get the pluginId of the applet? KPackage::metadata() is deprecated. Perhaps the KJob should be able to tell me eventually?

3.) How can I get the path to the installed package to get the info for 2.

4.) Not sure about linking KNotifications here


Thanks,

Kai Uwe Broulik

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