[PATCH] Binary clock config revamp

William Egert begert at gmail.com
Tue Jan 22 22:42:21 CET 2008


Looks great to me.  I think someone already touched on this, but can't we
change the text to "Use timezone", then when the box is checked the timezone
selection gets un-greyed?

-Bill

2008/1/22 Andre Magalhaes <andrunko at gmail.com>:

> Ooops forgot to attach the patch :)
>
> On Jan 22, 2008 5:59 PM, Andre Magalhaes <andrunko at gmail.com> wrote:
> > Hi again :),
> >
> > So this time it was the binary clock.
> >
> > Here are the old and new config dialog images
> >
> > Old config dialog:
> > http://www.flickr.com/photos/andrunko/2213008136/
> > New config dialog:
> > http://www.flickr.com/photos/andrunko/2213008138/
> >
> > The patch also add the possibility to select to use the local timezone.
> > It also fixes some coding style :)
> >
> > What do you guys think?
> > Ok to commit?
> >
> > --
> > Andre Moreira Magalhaes (andrunko)
> > --------------------------------------------------------
> > Jabber: andrunko at gmail.com
> > MSN:   andremoreira at msn.com
> > Skype:  andrunko
> > Blog:    http://andrunko.blogspot.com
> >
>
>
>
> --
> Andre Moreira Magalhaes (andrunko)
> --------------------------------------------------------
> Jabber: andrunko at gmail.com
> MSN:   andremoreira at msn.com
> Skype:  andrunko
> Blog:    http://andrunko.blogspot.com
>
> _______________________________________________
> Panel-devel mailing list
> Panel-devel at kde.org
> https://mail.kde.org/mailman/listinfo/panel-devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/panel-devel/attachments/20080122/0fe1f8f4/attachment.html 


More information about the Panel-devel mailing list