Review Request: Add a link to the current desktop folder instead of an icon applet if the desktop is set to folderview on the "Add to Desktop" Kickoff action

[Ignat Semenov]

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

(Updated March 18, 2012, 6:38 a.m.)


Review request for Plasma and Aaron J. Seigo.


Changes
-------

use QMetaObject::indexOfSlot to check for addUrl in the target containment
    
This is useful when e.g. the target containmnet is a FolderView, and we need to add a link instead of an Icon applet on triggering the "Add to Desktop" action.


Description
-------

Currently, right-clicking a Kickoff/Classical application launcher entry and selectiong "Add to Desktop" puts an icon applet on the desktop. However, if the desktop is set to Folderview, it is more consistent to add a link to the currently displayed folder. This patch cheks if the "folderview" plugin is used in the desktop containment and performs a KIO::link() if that's the case.


Diffs (updated)
-----

  plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp 8db9655 

Diff: http://git.reviewboard.kde.org/r/103830/diff/


Testing
-------

Tested, works.


Thanks,

Ignat Semenov

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