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

Reply via email to