Review Request: Improved extender theming
Aaron J. Seigo
aseigo at kde.org
Sat Oct 18 21:33:03 CEST 2008
On Saturday 18 October 2008, Rob Scheepmaker wrote:
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.vidsolbach.de/r/222/
> -----------------------------------------------------------
the diff on reviewboard seems broken, so i'll just reply here instead...
> Summary
> -------
>
> This is an attempt to make the appearance of extenders look more like this
> mockup made by pinheiro:
the screenshot included looks very sweet. nice to see that in action and not
just a mockup now =)
> - Added a virtual enabledBorderForItem(ExtenderItem*) to Extender.
> Depending on the setting of appearance, and the position of the item, this
> returns which background borders to enable.
why is it virtual?
> - Changed Dialog to check if it contains an extender, and in this case
> automatically enable the correct borders and set the correct margins of
> itself.
would be interested in seeing this part of the patch in particular ..
--
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 Qt Software
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20081018/fd82dd7c/attachment.sig
More information about the Plasma-devel
mailing list