Review Request 110334: %time Variable to get the current UTC time in the away message

[David Edmundson]

> On May 9, 2013, 10:38 p.m., Martin Klapetek wrote:
> > Looks very nice. One thing I realized though while trying this, is that we need to append the "UTC" somewhere in there, otherwise it won't make any sense. 
> > 
> > The use case
> > 
> > Let's consider UTC time 15:30, in my timezone (GMT+2) that would be 17:30, in Indian timezone (GMT+5:30) that would be 21:00. So if my indian friend goes away, his presence will say eg. "I'm away from 15:30". Me reading it will automatically parse it as my timezone (because that's how we're used to understand the time we see around us), while in reality in my timezone it'd be 17:30.
> > 
> > So we have two options - either do something like "hh:mm UTC/GMT" or let's do "hh:mm tz" where tz is the user's timezone (so it will become "21:00 GMT+5:30". I'm not sure which one is better, so I'd be fine with either one. Oh and let's loose the seconds, that big granularity does not bring any advantage (it does not really matter to the user if it was hh:mm:07 or if it was hh:mm:51).
> > 
> > Finally, we should put an example in the tooltip, eg "..., for example 17:16 GMT".
> > 
> > Thanks for working on this!

UTC and GMT are not the same. (they are for 6 months, but not during summer) 

So to clarify what you're saying, we should replace %time with

i18n("%1 UTC").arg( QDateTi... );


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110334/#review32295
-----------------------------------------------------------


On May 9, 2013, 3:19 p.m., Lucas Betschart wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110334/
> -----------------------------------------------------------
> 
> (Updated May 9, 2013, 3:19 p.m.)
> 
> 
> Review request for Telepathy and Daniele E. Domenichelli.
> 
> 
> Description
> -------
> 
> I've added an %time option which will get replaced by the current UTC time.
> 
> To make this option visible I've changed the tool tip (I think the first thing a user would do if he wants no away message is to remove the message text [if there is one, which isn't by default], so this double information can be replaced).
> 
> Adding an extra widget for drag n drop like in now playing would be too much for this I think. It would blow up the UI if I add this under every lineedit (as more stuff is visible on the first sight for the user as more complicated it looks) and it would be confusing if I add it only once in a central place, so I decided to just inform the user through a tool tip (it's also not a important feature, so it's ok if it's a little bit hidden).
> 
> 
> This addresses bug https://bugs.kde.org/show_bug.cgi?id=300849.
>     http://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=300849
> 
> 
> Diffs
> -----
> 
>   autoaway.cpp 06cb8e05d84f250e59a373461cb709ec5d400471 
>   config/telepathy-kded-config.cpp 780a882858c1fed4968b71cf4c5832f9d43f7b7a 
>   screensaveraway.cpp c7fdd75aa2217a44ec849dc9acfbf20020524bdb 
> 
> Diff: http://git.reviewboard.kde.org/r/110334/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Lucas Betschart
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130512/ef249cfe/attachment.html>