Review Request: Turn the LayoutWidget in the Tasks applet into a QGraphicsGridLayout subclass

Christian Mollekopf chrigi_1 at fastmail.fm
Wed Feb 18 20:10:26 CET 2009


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

Review request for Plasma, Aaron Seigo and Marco Martin.


Summary
-------

Since the gridlayout got fixed in qt 4.5 it's not anymore necessary to recreate the layout every time.
Therefore this patch turns the LayoutWidget into a QGraphicsGridLayout (which was my original intetion).
This should make it also easier to implement the popup with real items instead of the current implementation.
I'm actually not sure if the call to LayoutWidget::layoutItems from the TaskGroupItem is the perfect solution but it's necessary because the rows are not recalculated otherwise.


Diffs
-----

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

Diff: http://reviewboard.kde.org/r/122/diff


Testing
-------

Tested locally, no backdrafts found


Thanks,

Christian



More information about the Plasma-devel mailing list