From kde-panel-devel Fri Nov 26 00:12:31 2010 From: "Aaron Seigo" Date: Fri, 26 Nov 2010 00:12:31 +0000 To: kde-panel-devel Subject: Re: Review Request: Kickoff application uninstaller Message-Id: <20101126001231.30336.43412 () vidsolbach ! de> X-MARC-Message: https://marc.info/?l=kde-panel-devel&m=129073038505298 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============0789014653==" --===============0789014653== Content-Type: multipart/alternative; boundary="===============4399891478307832856==" --===============4399891478307832856== 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://svn.reviewboard.kde.org/r/5974/#review8981 ----------------------------------------------------------- Ship it! a comment in the code explaining why an activatable is being checked for wo= uld be nice. - Aaron On 2010-11-25 23:48:05, Daniel Nicoletti wrote: > = > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/5974/ > ----------------------------------------------------------- > = > (Updated 2010-11-25 23:48:05) > = > = > Review request for Plasma. > = > = > Summary > ------- > = > Patch to use PackageKit session interface to "Uninstall" the applications. > When kickoff starts it checks if the session interface is available, if i= t's not available the > right click option to "Uninstall" applications will not be displayed. > = > = > Diffs > ----- > = > /trunk/KDE/kdebase/workspace/plasma/desktop/applets/kickoff/ui/contextm= enufactory.cpp 1200785 = > = > Diff: http://svn.reviewboard.kde.org/r/5974/diff > = > = > Testing > ------- > = > = > Screenshots > ----------- > = > Kickoff Uninstaller > http://svn.reviewboard.kde.org/r/5974/s/563/ > = > = > Thanks, > = > Daniel > = > --===============4399891478307832856== 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://svn.reviewb= oard.kde.org/r/5974/

Ship it!

a comment =
in the code explaining why an activatable is being checked for would be nic=
e.

- Aaron


On November 25th, 2010, 11:48 p.m., Daniel Nicoletti wrote:

Review request for Plasma.
By Daniel Nicoletti.

Updated 2010-11-25 23:48:05

Descripti= on 

Patch to use PackageKit session interface to "Uninstall=
" the applications.
When kickoff starts it checks if the session interface is available, if it&=
#39;s not available the
right click option to "Uninstall" applications will not be displa=
yed.

Diffs=

  • /trunk/KDE/kdebase/workspace/plasma/desktop/applets/kickoff/ui/context= menufactory.cpp (1200785)

View Diff

Screensho= ts

3D"Kickof=
--===============4399891478307832856==-- --===============0789014653== 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 --===============0789014653==--