[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