----------------------------------------------------------- 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
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel