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