Review Request 119927: Port systemloadviewer plasmoid

[Martin Yrjölä]

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



File Attachment: Screenshot - 2014-08-24-212539_505x860_scrot.png
<https://git.reviewboard.kde.org//r/119927/#fcomment256>

    Yeah, better to have it consistent. I liked Anditosan's mockup for the new system settings. The color chooser looked like this http://wstaw.org/m/2014/08/25/shareimage-S11786.png
    
    So maybe something like the default combobox would be best? Here's the whole mockup https://dribbble.com/shots/1378722-MB-Class-Edit/attachments/198336



File Attachment: Screenshot - 2014-08-24-212539_505x860_scrot.png
<https://git.reviewboard.kde.org//r/119927/#fcomment257>

    I actually changed this string after the screenshot to "Use circular monitors", but you're right. Radio buttons or a combo box would be better. I'm not really sure if I should make the horizontal bars an option or just make them horizontal automatically if the plasmoid detects it's in a vertical panel. Any thoughts about that?


- Martin Yrjölä


On Aug. 24, 2014, 7:26 p.m., Martin Yrjölä wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119927/
> -----------------------------------------------------------
> 
> (Updated Aug. 24, 2014, 7:26 p.m.)
> 
> 
> Review request for Plasma and David Edmundson.
> 
> 
> Repository: kdeplasma-addons
> 
> 
> Description
> -------
> 
> Port systemloadviewer plasmoid
> 
> It had a VDG makeover and now there's also circular monitors available in the options.
> 
> There are regressions as well. These features aren't yet implemented:
> - Horizontal bar support
> - Individual monitors for each cpu
> 
> Probably there are some optimizations to be done. I'm not really sure how I should handle the two types of monitors (bar, circular) correctly. Now I just hide the one that is not enabled in the options.
> 
> 
> Diffs
> -----
> 
>   applets/CMakeLists.txt ccbe440342c7fc0ca5c9f67bcaa4e823cd35a099 
>   applets/systemloadviewer/CMakeLists.txt 6de867aa63e5102a5085667f8106ec09da2c4968 
>   applets/systemloadviewer/Messages.sh b95833b3b5cf7865e72ed6b46e8dcd7848c599e1 
>   applets/systemloadviewer/TODO 210dcd76f02b2a8f62e2e7339d089c545a4fb112 
>   applets/systemloadviewer/coloursconfig.ui 70a08afbaa204bb78865a545965210f3e2dfe42a 
>   applets/systemloadviewer/generalconfig.ui 2ae6d6540a12203e73bab5ca930da751f94bde22 
>   applets/systemloadviewer/package/contents/config/config.qml PRE-CREATION 
>   applets/systemloadviewer/package/contents/config/main.xml PRE-CREATION 
>   applets/systemloadviewer/package/contents/ui/BarMonitor.qml PRE-CREATION 
>   applets/systemloadviewer/package/contents/ui/CircularMonitor.qml PRE-CREATION 
>   applets/systemloadviewer/package/contents/ui/ColorPicker.qml PRE-CREATION 
>   applets/systemloadviewer/package/contents/ui/ConditionallyRoundedRectangle.qml PRE-CREATION 
>   applets/systemloadviewer/package/contents/ui/GeneralSettings.qml PRE-CREATION 
>   applets/systemloadviewer/package/contents/ui/SystemLoadViewer.qml PRE-CREATION 
>   applets/systemloadviewer/plasma-applet-systemloadviewer.desktop 9324d4216e716b2156f5f084c8b4782a6dd84806 
>   applets/systemloadviewer/systemloadviewer.h 9bbcf2305ef47dd8883600002a9b65954ba65a74 
>   applets/systemloadviewer/systemloadviewer.cpp bff0b34aa2d7a46e8e1fa052bb6008cf52a4497f 
> 
> Diff: https://git.reviewboard.kde.org/r/119927/diff/
> 
> 
> Testing
> -------
> 
> Works nicely on Air, Breeze light and Breeze dark Plasma themes. The Bar monitor is not pixel perfect because of the edge roundings. I haven't been able to test if this plasmoid works with high DPI monitors.
> 
> 
> File Attachments
> ----------------
> 
> Screenshot
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/08/24/a2a69145-6d76-48e8-bccd-ce7cead244bb__2014-08-24-212539_505x860_scrot.png
> 
> 
> Thanks,
> 
> Martin Yrjölä
> 
>

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