Review Request: Cleanups in pager

Anthony Bryant antjbryant at gmail.com
Wed Sep 15 05:05:04 CEST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5355/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

* Make configAccepted() just write the new config, and make configChanged() conditionally update the plasmoid after reading the config back in.
* Split the grid resizing logic into its own method: recalculateGridSizes(int rows), and make it detect more edge cases.
* Rename the main size calculation method to updateSizes(bool allowResize), which will resize the applet to its preferred size if allowResize is true.
* Fix a bug which caused applet resizes to be wrongly cancelled in some situations.
* Code style cleanups, remove unused variables, etc.

(Sorry for putting so many changes into one patch)


This addresses bug 250756.
    https://bugs.kde.org/show_bug.cgi?id=250756


Diffs
-----

  /trunk/KDE/kdebase/workspace/plasma/desktop/applets/pager/pager.h 1175398 
  /trunk/KDE/kdebase/workspace/plasma/desktop/applets/pager/pager.cpp 1175398 

Diff: http://svn.reviewboard.kde.org/r/5355/diff


Testing
-------

Tested configuring the pager from the config interface and the desktop console, and that it responds correctly to changes in desktop count, and in FormFactor and size.


Thanks,

Anthony

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100915/dcdfad0a/attachment.htm 


More information about the Plasma-devel mailing list