<table><tr><td style="">rkflx requested changes to this revision.<br />rkflx added a comment.<br />This revision now requires changes to proceed.
</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/D12321">View Revision</a></tr></table><br /><div><div><p>The relation between the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Preview</span></span></span> and the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Zoom</span></span></span> slider is now a bit weird: You can override with the button, but once you move the slider the setting will be gone again. Also, when showing the dialog after closing it, the slider has preference over the button. In addition, even though the button says "enabled", users will wonder why no thumbnails are showing!</p>

<p>Would it make sense to both:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Set the button state to "off" once the icons are too small, to make it clear that no previews are to be expected.</li>
<li class="remarkup-list-item">Disable the button, so manually toggling thumbnails only works if icons are big enough.</li>
</ul>

<p>Allowing to override even for small images would quickly lead to some kind of manual vs. automatic setting, but this would be even more confusing, so I'd rather not go in that direction.</p>

<hr class="remarkup-hr" />

<p>Also, before this goes in I'd like to see a similar patch for Dolphin. Not so much because of any implemention issue, but to get the (not quite uncontroversial) idea more exposure and to agree on a common vision for how thumbnails should behave everywhere.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12321">https://phabricator.kde.org/D12321</a></div></div><br /><div><strong>To: </strong>anemeth, VDG, Frameworks, ngraham, rkflx<br /><strong>Cc: </strong>rkflx, ngraham, Frameworks, michaelh, bruns<br /></div>