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

Reply via email to