Review Request: LayoutWidget overhaul and some other stuff

Christian Mollekopf chrigi_1 at hotmail.com
Sat Oct 25 15:04:45 CEST 2008


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.vidsolbach.de/r/241/
-----------------------------------------------------------

(Updated 2008-10-25 06:04:45.282789)


Review request for Plasma and Aaron Seigo.


Summary
-------

Hopefully the last major overhaul of the LayoutWidget, i put the calculation of the coumns and the rows together since they are logically tied and removed a lot of unused or unecessary stuff. Renamed all fillRows stuff to forceRows. Further i implemented some vertical layout stuff so it should be basically working now. Also the automatic calculation of the columns is now implemented though not yet perferct.


Diffs
-----

  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/layoutwidget.h
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/layoutwidget.cpp
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.h
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.cpp
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/tasks.cpp

Diff: http://reviewboard.vidsolbach.de/r/241/diff


Testing
-------

just basic testing. i tried it shortly.


Thanks,

Christian



More information about the Plasma-devel mailing list