[PATCH] Standardize clocks config dialogs

Andre Magalhaes andrunko at gmail.com
Mon Jan 21 17:26:11 CET 2008


Resending as the first one did went trough as the attachments
were big :).

2008/1/20 Aaron J. Seigo <aseigo at kde.org>:
> please post a screenshot so everyone can see the visual differences. this is
> what we encourage for visual changes.
Old config:
http://www.flickr.com/photos/andrunko/2209737372/
New config appearance tab:
http://www.flickr.com/photos/andrunko/2209737378/
New config timezones tab:
http://www.flickr.com/photos/andrunko/2209737382/

The new config is based on the digital-clock config. I believe all
clocks config should use the same layout, and should add the possibility
to use the local timezone.
I also changed the strings "Show seconds", "Display seconds" to be the same
across the clocks config.
IMHO the binary-clock config dialog on kdereview needs some love also, before
it moves to extragear :). See mail on kde-core-devel

> btw, you can also do ui.localTimeZone->checkState() == Qt::Checked as
> ui.localTimeZone->isChecked()  ..
I just used the same code as in digital-clock, should I change both?

> good job on following the coding style,
> though; patch looks rather clean =)
Tnx :)

-- 
Andre Moreira Magalhaes (andrunko)
--------------------------------------------------------
Jabber: andrunko at gmail.com
MSN:   andremoreira at msn.com
Skype:  andrunko
Blog:    http://andrunko.blogspot.com


More information about the Panel-devel mailing list