<table><tr><td style="">romangg accepted this revision.<br />romangg added a comment.<br />This revision is now accepted and ready to land.
</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/D6764" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Looks good to me now. The 0.71 factor is of course rather arbitrary, but I think just multiplying with an arbitrary fixed factor is in the context of the current Digital Clock code the best thing to do. Before commiting please also add a small comment, that this value was only chosen through testing, such that the size was fine. This way somebody, who later reads the code, does not think that there is some deep meaning to this magic number.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>BRANCH</strong><div><div>clock3</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6764" rel="noreferrer">https://phabricator.kde.org/D6764</a></div></div><br /><div><strong>To: </strong>januz, Plasma, VDG, romangg<br /><strong>Cc: </strong>romangg, mmustac, IlyaBizyaev, davidedmundson, ngraham, Zren, mart, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>