Review Request 124449: [digital-clock] Introduce new layouting state for small panels

[Heiko Tietze]

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

Ship it!


Yes, the pipe is less obtrusive and well-known as separator. Nice solution.

- Heiko Tietze


On Juli 24, 2015, 10:27 vorm., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124449/
> -----------------------------------------------------------
> 
> (Updated Juli 24, 2015, 10:27 vorm.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Bugs: 348072
>     https://bugs.kde.org/show_bug.cgi?id=348072
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> One of the most reported bugs wrt digital clock is that the second line
> text (date or timezone) is unreadable on small panels so this adds new
> state to the clock applet that lays things horizontally when the panel
> is too small to show text with the "small" font (set in system settings).
> 
> When there is a date displayed, a delimiter is put in between to allow
> easier and faster spotting the time (as on a first glance there is just
> a bunch of numbers, so quickly orienteering is super important). I used
> the dot symbol first but then went for pipe which I think looks more clean;
> see screenshots for comparison.
> 
> I've made a quick video showing the state changes,
> see https://www.youtube.com/watch?v=t7w0k8H22h4 - it shows the bullet
> delimiter and a bit wrong spacing which is now fixed.
> 
> 
> Diffs
> -----
> 
>   applets/digital-clock/package/contents/ui/DigitalClock.qml 9506078 
> 
> Diff: https://git.reviewboard.kde.org/r/124449/diff/
> 
> 
> Testing
> -------
> 
> Everything works as it used and now with small panel things are layed
> horizontally, see video above.
> 
> 
> File Attachments
> ----------------
> 
> Delimiters
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/07/24/e0aa0799-ca56-4bdb-b4e6-b8db6622ff10__clock-delimiters.png
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

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