Review Request: Quicklaunch applet: Change icon order

Lukas Appelhans l.appelhans at gmx.de
Wed Mar 10 23:42:46 CET 2010


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


Looks quite fine to me :) Just minor issues...


/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunchApplet.cpp
<http://reviewboard.kde.org/r/3254/#comment3932>

    No need for the QGraphicsGridLayout:: in there...



/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunchApplet.cpp
<http://reviewboard.kde.org/r/3254/#comment3933>

    again no need for QGraphicsGridLayout:: :)


- Lukas


On 2010-03-10 22:32:10, Ingomar Wesp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3254/
> -----------------------------------------------------------
> 
> (Updated 2010-03-10 22:32:10)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Since there were no objections on plasma-devel, I'm now posting a review request for a patch that changes the icon order in the quicklaunch applet.
> 
> Currently, icons are arranged in column major order. Thus, when 5 icons are 
> displayed in two rows, the indices are distributed as follows:
> 
> +---+---+---+
> | 0 | 2 | 4 |
> +---+---+---+
> | 1 | 3 |
> +---+---+
> 
> This patch changes the arrangement to row major order instead:
> 
> +---+---+---+
> | 0 | 1 | 2 |
> +---+---+---+
> | 3 | 4 |
> +---+---+
> 
> Additionally, I have removed QuicklaunchLayout::addItem(...). Its only caller - QuicklaunchApplet::performUiRefactor() - already knows the number of rows and columns, so there is no need to recompute them. In the long term, I'd like to remove QuicklaunchLayout and move layouting and drag/drop into a dedicated widget class (QuicklaunchIconGrid maybe).
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/QuicklaunchLayout.h 1100817 
>   /trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/QuicklaunchLayout.cpp 1100882 
>   /trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunchApplet.cpp 1100902 
> 
> Diff: http://reviewboard.kde.org/r/3254/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ingomar
> 
>



More information about the Plasma-devel mailing list