<table><tr><td style="">januz 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/D6764" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I think this explains why you haven't had a review, this existing code is already sketchy as-is :/<br />
 But we should be trying to fix that not just all of us ignoring it. Sorry about that.</p></blockquote>

<p>No problem, I can understand the dread of piling stuff on code that needs refactoring.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>So this clock has 4 states:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">horizontal</li>
<li class="remarkup-list-item">horizontal small (date and tz appear alongside clock)</li>
<li class="remarkup-list-item">vertical</li>
<li class="remarkup-list-item">other ()</li>
</ol></blockquote>

<p>I don't think I've ever seen state 4. How do you trigger it?</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>With your patch:<br />
 in states 1 and 2 you're explicitly setting the time zone label to a config value, and implicitly the date and TZ label to 0.7 of that.</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);">I could understand logic that limits the maximiumSize to a user defined value, but I don't want to be in a position where the user resizes a panel..then has to open all the config options of each widget and adjust things.
Would keeping the fit mode but setting the maximumPixelSize to the config value work for you?</pre></div></blockquote>

<p>That's a better solution! We could use a float value, and present it more as a percentage instead of a font size value.<br />
The only thing to look out for is a minimum size limit, so the widget doesn't vanish when the user resizes the panel/widget.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>States 3 and 4 are unchanged:</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);">I don't really want to have a config option that simply does nothing. Is there a reason it's not changed?</pre></div></blockquote>

<p>The main idea of this setting was to be able to make the clock smaller than the panel size. Vertical panels are usually thin, and the font size for digital clocks turns out to be fairly small already. So I didn't think it was really useful to set a font size there. It wouldn't hurt either though.</p></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<br /><strong>Cc: </strong>davidedmundson, ngraham, Zren, mart, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>