----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123274/ -----------------------------------------------------------
(Updated April 6, 2015, 4:05 nachm.) Review request for Plasma. Changes ------- Rename General to Advanced and make "Appearance" the first tab shown when opening the config dialog Repository: kdeplasma-addons Description (updated) ------- Added tab "General" and "Appearance" in the config ui General contains the bits about command execution after the timer has completed Appearance contains the ability to hide seconds and showing the title The configuration option for showing the title is a bit buggy (again) This bugreport explains the wrong scaling of the title: 304923 I've converted this plasmoid from being qml only to c++ and qml because it looks like KProcess has no qml interface yet (and I didn't find any place where to start) [Edit 1]: updated to use QProcess [Edit 2]: renamed General to Advanced and moved it down to make "Appearance" the default tab on opening Diffs (updated) ----- applets/timer/plugin/timerplugin.cpp PRE-CREATION applets/timer/plugin/timer.cpp PRE-CREATION applets/timer/plugin/timerplugin.h PRE-CREATION applets/timer/plugin/timer.h PRE-CREATION applets/timer/plugin/qmldir PRE-CREATION applets/timer/package/contents/ui/timer.qml f675b93bda59eb161fe10b5f575eaa2082b822e0 applets/timer/package/contents/ui/configAppearance.qml PRE-CREATION applets/timer/package/contents/ui/configAdvanced.qml PRE-CREATION applets/timer/package/contents/config/config.qml PRE-CREATION applets/timer/CMakeLists.txt c7cba10a96e6d3f1ba98a3aa755d6d3d8aad0c9f Diff: https://git.reviewboard.kde.org/r/123274/diff/ Testing ------- Tested using plasmashell directly and using plasmoidviewer File Attachments (updated) ---------------- Appearance Tab https://git.reviewboard.kde.org/media/uploaded/files/2015/04/06/b9a4c7dd-c272-4b4c-807b-145102e17d13__tab_appearance.png Advanced Tab https://git.reviewboard.kde.org/media/uploaded/files/2015/04/06/ca6d1f98-70fe-4d13-a170-ec44cb82bce4__tab_advanced.png Thanks, Bernhard Friedreich
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
