Review Request: Proper size for analog clock timezone label
Alain Boyer
alainboyer at gmail.com
Mon Jun 21 07:16:52 CEST 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4418/
-----------------------------------------------------------
Review request for Plasma.
Summary
-------
This patch adjusts the timezone label of the analog clock to the width of the city text. Although this is a cosmetic change that is very subjective, I find that the overall look, when multiple clocks are horizontally aligned next to each other, is nicer and more polished.
This is a very minor change, but since I have been away from KDE hacking for a while and we are so close to the next release, I figured I'd put it up for review.
Diffs
-----
/trunk/KDE/kdebase/workspace/plasma/generic/applets/analog-clock/clock.h 1140389
/trunk/KDE/kdebase/workspace/plasma/generic/applets/analog-clock/clock.cpp 1140389
Diff: http://reviewboard.kde.org/r/4418/diff
Testing
-------
Works just fine in plasmoidviewer.
Screenshots
-----------
Before
http://reviewboard.kde.org/r/4418/s/440/
After
http://reviewboard.kde.org/r/4418/s/441/
Thanks,
Alain
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100621/4e348805/attachment.htm
More information about the Plasma-devel
mailing list