Review Request: Fix pager applet for vertical panels

michael-olbrich at web.de michael-olbrich at web.de
Sat Jul 25 03:13:24 CEST 2009


-----------------------------------------------------------
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



More information about the Plasma-devel mailing list