----------------------------------------------------------- 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 _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
