Review Request: [Quicklaunch] Refactoring of the Quicklaunch applet
Ingomar Wesp
ingomar at wesp.name
Fri Apr 23 21:08:34 CEST 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3786/
-----------------------------------------------------------
Review request for Plasma.
Summary
-------
This is my proposed patch for the refactored quicklaunch applet as discussed in review request http://reviewboard.kde.org/r/3358/ and on the ML a while ago. Sorry that it took so long...
Summary of changes:
- Refactored the quicklaunch applet, so that the applet,
icon grid widget and icon grid layout are split into
separate classes all living in a newly created namespace.
- Improved drag & drop behaviour (it is not possible to drop items
in the popup dialog) and drag & drop markers.
- Icons are now moved to/from the dialog explicitly instead of
asking the user to specify the number of the icons that are
shown in the primary area.
- Icon size is now determined automatically based on the
available space, hard-coded minimum and maximum bounds and
the number of rows (or columns) set by the user. This is done
in a custom layout that is no longer based on
QGraphicsGridLayout.
- When all icons are removed from an icon area, a placeholder
icon is displayed.
As this patch changes the configuration keys used, it also incorporates code for migrating older config keys.
Unfortunately, using svn diff with files that have been "svn move"d appears to yield broken diffs, so the patch here does not reflect the history for files that are based on renamed files (quicklaunch.*, quicklaunchicon.*), but I'll make sure that the history is preserved when committing (if this gets a "ship it", that is).
Please give it a spin and tell me what you think.
This addresses bugs 206382, 206912, 214463, 225011, and 233914.
https://bugs.kde.org/show_bug.cgi?id=206382
https://bugs.kde.org/show_bug.cgi?id=206912
https://bugs.kde.org/show_bug.cgi?id=214463
https://bugs.kde.org/show_bug.cgi?id=225011
https://bugs.kde.org/show_bug.cgi?id=233914
Diffs
-----
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/CMakeLists.txt 1117710
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/QuicklaunchLayout.h 1117710
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/QuicklaunchLayout.cpp 1117710
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/icongrid.h PRE-CREATION
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/icongrid.cpp PRE-CREATION
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/icongridlayout.h PRE-CREATION
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/icongridlayout.cpp PRE-CREATION
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunch.h PRE-CREATION
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunch.cpp PRE-CREATION
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunchApplet.h 1117710
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunchApplet.cpp 1117710
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunchConfig.ui 1117710
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunchIcon.h 1117710
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunchIcon.cpp 1117710
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunchicon.h PRE-CREATION
/trunk/KDE/kdebase/workspace/plasma/generic/applets/quicklaunch/quicklaunchicon.cpp PRE-CREATION
Diff: http://reviewboard.kde.org/r/3786/diff
Testing
-------
Thanks,
Ingomar
More information about the Plasma-devel
mailing list