Review Request 114110: Prints time in the lower right corner

Marco Martin notmart at gmail.com
Mon Nov 25 15:19:54 UTC 2013


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


I'm not really the reviewer for it, but just my 2 cents:

that's pretty much the reason why at the beginning of the project (a lot of years ago by now;) we wanted the possibility to have plasmoids around in the media center.

Just hardcoding the time, while seems easy to do, opens a can of worms since being 12h or 24 h must have to depend from the system settings, then many will want it, many will not want it, then someone may want to have the date too.. and that pretty much duplicates the clock plasmoid featuers ;)

- Marco Martin


On Nov. 25, 2013, 3:12 p.m., Oskar Jauch wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/114110/
> -----------------------------------------------------------
> 
> (Updated Nov. 25, 2013, 3:12 p.m.)
> 
> 
> Review request for Plasma and Shantanu Tushar.
> 
> 
> Repository: plasma-mediacenter
> 
> 
> Description
> -------
> 
> Shows time in the lower right corner in 12 hours format in the Plasma MediaCenter.
> 
> 
> Diffs
> -----
> 
>   mediaelements/mediawelcome/HomeScreenHeader.qml 2127ce7 
>   mediaelements/mediawelcome/HomeScreenFooter.qml e6173de 
> 
> Diff: http://git.reviewboard.kde.org/r/114110/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Oskar Jauch
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20131125/82ee1959/attachment.html>


More information about the Plasma-devel mailing list