----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119927/#review65326 -----------------------------------------------------------
Ship it! applets/systemloadviewer/package/contents/ui/ConditionallyLoadedMonitors.qml <https://git.reviewboard.kde.org/r/119927/#comment45647> For bonus points make it load nothing when it's not visible. It'll save a bit more. applets/systemloadviewer/package/contents/ui/ConditionallyLoadedMonitors.qml <https://git.reviewboard.kde.org/r/119927/#comment45646> personally I would have reduced it to. source: switch( monitorType) { case 0: .... return "foo"; } Bindings are generally better than functions as they can be optimised by the QML engine. not saying yours is wrong, but it's something to think about in future. - David Edmundson On Aug. 26, 2014, 9:43 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. 26, 2014, 9:43 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/systemloadviewer.h > 9bbcf2305ef47dd8883600002a9b65954ba65a74 > applets/systemloadviewer/systemloadviewer.cpp > bff0b34aa2d7a46e8e1fa052bb6008cf52a4497f > > applets/systemloadviewer/package/contents/ui/ConditionallyLoadedMonitors.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/package/contents/ui/CircularMonitor.qml > PRE-CREATION > applets/systemloadviewer/package/contents/ui/ColorPicker.qml PRE-CREATION > > 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ä > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel