<table><tr><td style="">kossebau updated this revision to Diff 55777.<br />kossebau added a comment.
</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/D20112">View Revision</a></tr></table><br /><div><div><p>Instead of using the size of a hint to define the rotation center of a hand<br />
shape, use the relative position of the (center of) hint.<br />
This allows to define rotation centers outside of the hand element, like<br />
useful for all those themes which use hands shapes that do not cover the<br />
clock center.<br />
It also is visually for the designer more logically to place e.g. a circle<br />
as rotation center hint exactly where the center of rotation is for a hand,<br />
instead of the abstract offset shape hint.<br />
And the relative positioning also solves the issue of having the rotation<br />
center directly at the border, where before the offset hint was not possibly<br />
due to a size of 0 in one dimension resulting in the hint not being visual<br />
and thus not propagated by QSvgRenderer/Plasma::Svg.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D20112?vs=55026&id=55777">https://phabricator.kde.org/D20112?vs=55026&id=55777</a></div></div><br /><div><strong>BRANCH</strong><div><div>fixclockshadowhandcenter</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20112">https://phabricator.kde.org/D20112</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/analog-clock/contents/ui/Hand.qml<br />
applets/analog-clock/contents/ui/analogclock.qml</div></div></div><br /><div><strong>To: </strong>kossebau, Plasma, VDG, mart, davidedmundson, ngraham<br /><strong>Cc: </strong>plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>