Review Request: Quicklaunch applet: Partial fix for drag&drop to main layout.

Lukas Appelhans l.appelhans at gmx.de
Mon Mar 8 22:52:04 CET 2010


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


Please take care of the coding style at some parts... for example row+1 -> row + 1, else { -> } else {...

Except that: If it works it should be fine :)

Lukas

- Lukas


On 2010-03-08 21:32:38, Ingomar Wesp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3214/
> -----------------------------------------------------------
> 
> (Updated 2010-03-08 21:32:38)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> This patch changes the computation of the insert position when dropping something on the applet's layout (it does not address dropping to the dialog yet). Additionally, appending items to the end of the list should be fixed now (it lead to a crash previously).
> 
> If I'm not mistaken, this should fix bugs #200798 and #225011.
> 
> 
> This addresses bugs 200798 and 225011.
>     https://bugs.kde.org/show_bug.cgi?id=200798
>     https://bugs.kde.org/show_bug.cgi?id=225011
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunchApplet.cpp 1100817 
> 
> Diff: http://reviewboard.kde.org/r/3214/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ingomar
> 
>



More information about the Plasma-devel mailing list