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 20, 2012, 5:05 p.m.)


Review request for Plasma and Aaron J. Seigo.


Changes
-------

addUrl(KUrl) -> addUrls(const KUrl::List&)

ReviewBoard and multiple git repos for different parts of kde is not the most comfortable thing to use, though..


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 99c2649 

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/20120320/fc5edc87/attachment.html>