--===============3772704435931641399== Content-Type: multipart/alternative; boundary="===============6787335290099119554==" --===============6787335290099119554== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103830/#review11978 ----------------------------------------------------------- This review has been submitted with commit 4d3df108a9a8b7431aff5cc54a78efcc= 89cbcf5d by Ignat Semenov to branch master. - Commit Hook On March 20, 2012, 5:05 p.m., Ignat Semenov wrote: > = > ----------------------------------------------------------- > 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. > = > = > 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" plu= gin is used in the desktop containment and performs a KIO::link() if that's= the case. > = > = > Diffs > ----- > = > plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp 99c2649 = > = > Diff: http://git.reviewboard.kde.org/r/103830/diff/ > = > = > Testing > ------- > = > Tested, works. > = > = > Thanks, > = > Ignat Semenov > = > --===============6787335290099119554== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable

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

This revie=
w has been submitted with commit 4d3df108a9a8b7431aff5cc54a78efcc89cbcf5d b=
y Ignat Semenov to branch master.

- Commit

On March 20th, 2012, 5:05 p.m., Ignat Semenov wrote:

Review request for Plasma and Aaron J. Seigo.

By Ignat Semenov.

Updated March 20, 2012, 5:05 p.m.

Descripti= on

Currently, right-clicking a Kickoff/Classical application la=
uncher entry and selectiong "Add to Desktop" puts an icon applet =
on the desktop. However, if the desktop is set to Folderview, it is more co=
nsistent 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.

Testing <= /h1>

Tested, works.

Diffs=

plasma/desktop/applets/kickoff/ui/contextmenufactory.cpp (99c2649)

View Diff

--===============6787335290099119554==-- --===============3772704435931641399== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel --===============3772704435931641399==--