Review Request: Fix FlowLayout to work.

Aaron Seigo aseigo at kde.org
Sat Feb 9 04:25:37 CET 2008


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://matt.rogers.name/r/96/#review88
-----------------------------------------------------------

Ship it!


that is remarkably little change to get that working properly. nicely done =)

please comment out (not remove, just comment it out) the debug output before you commit though so we don't have yet more noisy output (we already have enough ;)


- Aaron


On 2008-02-08 19:48:31, Bill Egert wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://matt.rogers.name/r/96/
> -----------------------------------------------------------
> 
> (Updated 2008-02-08 19:48:31)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Initial Re-work of the relayout() function so that it actually works.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/libs/plasma/layouts/flowlayout.cpp
> 
> Diff: http://matt.rogers.name/r/96/diff
> 
> 
> Testing
> -------
> 
> changed taskgroupitem in tasks applet to use flowlayout instead of BoxLayout.  With this change, I tested starting multiple apps and resizing the applet on the desktop.
> 
> 
> Thanks,
> 
> Bill
> 
>



More information about the Panel-devel mailing list