[Panel-devel] [PATCH] Beginnings of a panel implementation for discussion
Aaron J. Seigo
aseigo at kde.org
Fri Aug 31 22:18:55 CEST 2007
On Friday 31 August 2007, Robert Knight wrote:
> Updated version of the patch attached in response to initial review by
> richmoore.
looks mostly good =)
i suppose BoxLayout::expandingDirections() should really just return
d->expandingDirections()... or even better, get rid of
Private::expandingDirections and just always use
BoxLayout::expandingDirections()?
coding conventions aren't followed. e.g.:
+ if ( isExpanding )
+ sizes[i] = perItemSize;
+ else
+ sizes[i] = d->size(item->sizeHint());
should be:
+ if (isExpanding) {
+ sizes[i] = perItemSize;
+ } else {
+ sizes[i] = d->size(item->sizeHint());
+ }
i can't say i'm overly pleased about putting multiple classes in a single file
either, even if they are tiny.
it also references layoutAnimator which doesn't seem to be in the patch. i
guess you could commit it with that commented out before we move on to the
animation support you've done?
> - Basic testing with nested layouts.
as in "you've done basic testing" or "you have a basic test you want to commit
to svn"?
--
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43
KDE core developer sponsored by Trolltech
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20070831/1e6b4adb/attachment.pgp
More information about the Panel-devel
mailing list