Review Request 120002: Systemloadviewer: settings window redesign

Martin Yrjölä martin.yrjola at gmail.com
Sat Aug 30 13:17:21 UTC 2014


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

(Updated Aug. 30, 2014, 1:17 p.m.)


Review request for Plasma and David Edmundson.


Changes
-------

Get rid of the spacers, use i18nc to aid translation and add colon for consistency.


Repository: kdeplasma-addons


Description
-------

More polish by courtesy of the VDG.


Diffs (updated)
-----

  applets/systemloadviewer/package/contents/config/config.qml 570cc77ece3ed0bf84d89e45426773954f95d409 
  applets/systemloadviewer/package/contents/config/main.xml 5e38045eb369e029564abf973827eb84534eb9f4 
  applets/systemloadviewer/package/contents/ui/ColorPicker.qml 1b3c88233da75b9d001a0159dd4298d41f7d307c 
  applets/systemloadviewer/package/contents/ui/ColorSettings.qml PRE-CREATION 
  applets/systemloadviewer/package/contents/ui/GeneralSettings.qml fd719d5f2afed8c630fd6a1281f6ddda674b60c2 
  applets/systemloadviewer/package/contents/ui/SystemLoadViewer.qml 9e101f8fd68602495366c0dc8226927a572e0267 

Diff: https://git.reviewboard.kde.org/r/120002/diff/


Testing
-------

Mostly cosmetic changes, but I tested that the options does still work.


File Attachments
----------------

General settings
  https://git.reviewboard.kde.org/media/uploaded/files/2014/08/30/7ed9b718-93bd-4936-98dd-7a58b8b51128__2014-08-30-142153_361x169_scrot.png
Color settings with enabled manual color settings
  https://git.reviewboard.kde.org/media/uploaded/files/2014/08/30/8e2367fd-9f44-4bd2-87fe-df5ffbd746be__2014-08-30-143838_256x304_scrot.png
Color settings with disabled manual color settings
  https://git.reviewboard.kde.org/media/uploaded/files/2014/08/30/8399a4e1-7fa0-4369-ae30-cadbe455d2bb__2014-08-30-143853_254x304_scrot.png


Thanks,

Martin Yrjölä

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


More information about the Plasma-devel mailing list