Review Request: FIXME part of quicklaunch

Sujith H sujith.h at gmail.com
Mon May 4 21:00:56 CEST 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/649/
-----------------------------------------------------------

(Updated 2009-05-04 12:00:56.303439)


Review request for Plasma and Aaron Seigo.


Changes
-------

I am stuck to find out the position of the application to be inserted. The problem is in the 'else' part of the dropApp method.


Summary
-------

This patch is regarding the FIXME part of the quicklaunchApplet.cpp. I had added one more method named dropApp. Added the processDrop as per Aaron seigo. 


Diffs (updated)
-----

  kde/trunk/KDE/kdebase/workspace/plasma/applets/quicklaunch/workspace/plasma/applets/quicklaunch/quicklaunchApplet.h 963449 
  kde/trunk/KDE/kdebase/workspace/plasma/applets/quicklaunch/workspace/plasma/applets/quicklaunch/quicklaunchApplet.cpp 963449 

Diff: http://reviewboard.kde.org/r/649/diff


Testing
-------

Done basic testing and it works.


Thanks,

Sujith



More information about the Plasma-devel mailing list