[Panel-devel] BorderLayout

[Aaron J. Seigo]

On Saturday 15 September 2007, Ivan Čukić wrote:
> > these everywhere. currently we have Direction and (screen) Location;
> > adding position would round it out nicely.
>
> If you say so. I'm a bit worried that this (top, left...) is not what
> people will think of when they read Position (especially after QPos)

i don't think it's a problem =)

> > - needs a dptr =)
>
> Bummer. I was hoping never to hear those words -)

hehe ... well, libplasma does need to provide binary compat at SOME point. ;)

> > setSize(Plasma::Position pos, qreal size); or setSize(qreal size,
> > Plasma::Position pos). less API and somehow feels more "natural" to me
> > though that's 99% subjective =)
>
> Nice idea with a small adjustment - instead of just Plasma::Position, it
> could be bit-set of Plasma::Position so multiple positions can still be set
> at once.

hm... perhaps.

> > - why both m_items and m_itemPositions? i see that m_items is used for
> > itemAt(int) and is used to reflect the order of addition, however m_items
> > can easily end up with more than 5 items in it (call addItem repeatedly
> > =) even though only 5 items will be shown from what i see here ...
> > perhaps itemAt(int) should just index into m_itemPositions, implying a
> > connection between position and ordering? would lower the overhead and
> > avoid mess ups with m_items...
>
> I was thinking (and still am) to allow multiple widgets in one position.
> I'll see what comes of it.

hm.. wouldn't that be easier/better by simply adding a layout to that area?

-- 
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/20070915/d8addff0/attachment.pgp