Review Request: FIXME part of quicklaunch

Sujith H sujith.h at gmail.com
Mon May 4 20:55:22 CEST 2009


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

(Updated 2009-05-04 11:55:21.794595)


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 962341 
  /kde/trunk/KDE/kdebase/workspace/plasma/applets/quicklaunch/workspace/plasma/applets/quicklaunch/quicklaunchApplet.cpp 962341 

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