<table><tr><td style="">aloiseau created this revision.<br />aloiseau added reviewers: Plasma: Workspaces, kossebau.<br />aloiseau added a project: Plasma: Workspaces.<br />Herald edited projects, added Plasma; removed Plasma: Workspaces.<br />Herald added a subscriber: plasma-devel.<br />aloiseau requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D28262">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Add a configuration option to hide minute hand, the same way second hand can already be hidden.</p>

<p>Those two options are independent, letting user hide minute hand alone.</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);"></pre></div>

<p>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.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Add an Analog Clock to your Plasma wallpaper</li>
<li class="remarkup-list-item">Go to its setting</li>
<li class="remarkup-list-item">Deselect new "Show minute hand" option</li>
<li class="remarkup-list-item">Apply</li>
</ul>

<p>Minute hand is then hidden from Analog Clock widget.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28262">https://phabricator.kde.org/D28262</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/analog-clock/contents/config/main.xml<br />
applets/analog-clock/contents/ui/analogclock.qml<br />
applets/analog-clock/contents/ui/configGeneral.qml</div></div></div><br /><div><strong>To: </strong>aloiseau, Plasma: Workspaces, kossebau<br /><strong>Cc: </strong>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<br /></div>