D28262: [applets/analog-clock] Optional minute hand

[Anthony Loiseau]

aloiseau created this revision.
aloiseau added reviewers: Plasma: Workspaces, kossebau.
aloiseau added a project: Plasma: Workspaces.
Herald edited projects, added Plasma; removed Plasma: Workspaces.
Herald added a subscriber: plasma-devel.
aloiseau requested review of this revision.

REVISION SUMMARY
  Add a configuration option to hide minute hand, the same way second hand can already be hidden.
  
  Those two options are independent, letting user hide minute hand alone.
  
        
  
  Clock refresh interval is not modified. It still refreshes every 30 seconds when second hand is hidden, even when hour hand is the only visible one. Having this interval computed from both hiddable hands would make its computation less readable for a minor CPU benefit.

TEST PLAN
  - Add an Analog Clock to your Plasma wallpaper
  - Go to its setting
  - Deselect new "Show minute hand" option
  - Apply
  
  Minute hand is then hidden from Analog Clock widget.

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D28262

AFFECTED FILES
  applets/analog-clock/contents/config/main.xml
  applets/analog-clock/contents/ui/analogclock.qml
  applets/analog-clock/contents/ui/configGeneral.qml

To: aloiseau, #plasma_workspaces, kossebau
Cc: aloiseau, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200324/34bfa5c1/attachment.html>