<table><tr><td style="">ngraham accepted this revision.<br />ngraham added a reviewer: rkflx.<br />ngraham added a subscriber: rkflx.<br />ngraham 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/D12321">View Revision</a></tr></table><br /><div><div><p>Very nice, works great for me. Once this goes in, I think we can safely turn on previews by default, because we won't regress anything for people who prefer small icons who don't need (and probably don't like) previews. Adding <a href="https://phabricator.kde.org/p/rkflx/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@rkflx</a> as a reviewer since I know he feels strongly about previews. :)</p>

<p>In fact, we should really do this for Dolphin too, as currently previews are on by default but uselessly apply to tiny icons just like in the file dialogs before this patch. Wanna have a crack at that?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</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>