----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1123/ -----------------------------------------------------------
Review request for Plasma. Summary ------- This is an attempt to fix the pager applet for vertical panels. There are currently two problems: 1. The configuration file always contains the number of rows. This is shared between all pager applets. This makes it basically impossible to have a pager on both a vertical and a horizontal panel. One of the two is always too small. 2. The parameter for setPreferredSize() is wrong. As a result, a large part of the pager disappears behind the next applet. With this patch the config value contains the number of rows for horizontal panels and the number of columns for vertical panels. As a side effect, changing a panel from horizontal to vertical does now the right thing (imho): One row is turned into on column. Diffs ----- /trunk/KDE/kdebase/workspace/plasma/applets/pager/pager.cpp 1001981 Diff: http://reviewboard.kde.org/r/1123/diff Testing ------- Thanks, michael _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel