Review Request 128136: alignement for systemmonitoring applets

[Burkhard Lück]

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




applets/systemmonitor/common/contents/ui/DoublePlotter.qml (lines 36 - 37)
<https://git.reviewboard.kde.org/r/128136/#comment65243>

    Please use context e.g.
    
    i18nc("%1 and %3 are data values, %2 and %4 are unit datatypes", "%1 %2 | %3 %4", ...



applets/systemmonitor/cpu/contents/ui/cpu.qml (lines 42 - 44)
<https://git.reviewboard.kde.org/r/128136/#comment65241>

    Please use context like this:
    
    - i18nc("CPU usage: %1 is the value, %2 the unit datatype", "%1 %2")
    - return i18n("%1 %2", Math.round(data.value), data.units);
    + return i18nc("CPU usage: %1 is the value, %2 the unit datatype", "%1 %2", Math.round(data.value), data.units);



applets/systemmonitor/diskactivity/contents/ui/diskactivity.qml (lines 41 - 42)
<https://git.reviewboard.kde.org/r/128136/#comment65242>

    Please use context e.g.
    
    return i18nc("%1 and %1 are values of the same datatype", "%1/s | %2/s", ...


- Burkhard Lück


On Juni 16, 2016, 9:13 vorm., Andreas Kainz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128136/
> -----------------------------------------------------------
> 
> (Updated Juni 16, 2016, 9:13 vorm.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> 1. change the minimum size and the min width in the panel
> 2. speedLabels are now right align to the label cause you show the number which is drown on the right part of the diagramm 
> 3. round to 14 % instead of 14,14 % for cpu load
> 
> still missing:
> a. hide the heading if the desktop widget has to less space for the rows
> b. better friendlyName for ram, network
> c. remove the gray line from the chart (0-line) at least when the applet was shown on the panel
> d. don't show 0 KB/s or 0 B, ...
> 
> shedule
> thanks for your review
> if you know how I can fix one of the missing things, let me know
> tiny improvements when missing stuff is finished.
> 
> 
> Diffs
> -----
> 
>   applets/systemmonitor/common/contents/ui/Applet.qml fffd8e0 
>   applets/systemmonitor/common/contents/ui/DoublePlotter.qml 738ffbc 
>   applets/systemmonitor/common/contents/ui/SinglePlotter.qml 987fd2c 
>   applets/systemmonitor/cpu/contents/ui/cpu.qml 4ebbf92 
>   applets/systemmonitor/diskactivity/contents/ui/diskactivity.qml 994336f 
> 
> Diff: https://git.reviewboard.kde.org/r/128136/diff/
> 
> 
> Testing
> -------
> 
> on the desktop and the panel
> 
> 
> File Attachments
> ----------------
> 
> screenshot of the result
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/06/16/4fecb3bc-d540-4358-af1d-342c77b9cfa3__Screenshot_20160616_111131.png
> 
> 
> Thanks,
> 
> Andreas Kainz
> 
>

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