Review Request 119758: Add option to DigitalClock applet to show date

David Edmundson david at davidedmundson.co.uk
Wed Aug 13 11:00:32 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119758/#review64446
-----------------------------------------------------------



applets/digital-clock/package/contents/ui/DigitalClock.qml
<https://git.reviewboard.kde.org/r/119758/#comment45034>

    Do we really want the same text label with the same font?
    
    I'd like some screenshots.



applets/digital-clock/package/contents/ui/DigitalClock.qml
<https://git.reviewboard.kde.org/r/119758/#comment45032>

    In the size helper we calculate the size of 
    00:00:00 which is the widest string so that text doesn't jump about.
    
    Here you're working out the size of the current day. Which is going to be dynamic. Given this is only calculated once on startup, if you start it in May you'll get it broken if Plasma keeps running till September when the text will no longer fit.



applets/digital-clock/package/contents/ui/configAppearance.qml
<https://git.reviewboard.kde.org/r/119758/#comment45033>

    can be just:
    
    property string cfg_dateFormat: model[currentIndex]["name"]
    
    and remove the onCurrentIndexChanged


- David Edmundson


On Aug. 13, 2014, 10:51 a.m., Dan Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119758/
> -----------------------------------------------------------
> 
> (Updated Aug. 13, 2014, 10:51 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Bugs: 335006
>     https://bugs.kde.org/show_bug.cgi?id=335006
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This patch adds option to show date in the DigitalClock applet, something I was very much missing after switching to Plasma 5 :-)
> 
> Similary to the KDE 4 applet, users can choose from long, short and narrow date formats.
> 
> 
> Diffs
> -----
> 
>   applets/digital-clock/package/contents/config/main.xml 74ac640 
>   applets/digital-clock/package/contents/ui/DigitalClock.qml 20babc3 
>   applets/digital-clock/package/contents/ui/configAppearance.qml ab67e14 
> 
> Diff: https://git.reviewboard.kde.org/r/119758/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dan Vrátil
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140813/92112a02/attachment-0001.html>


More information about the Plasma-devel mailing list