D12321: Hide file preview when icon is too small

[Nathaniel Graham]

ngraham added a comment.


  In D12321#249999 <https://phabricator.kde.org/D12321#249999>, @xyquadrat wrote:
  
  > In D12321#249981 <https://phabricator.kde.org/D12321#249981>, @ngraham wrote:
  >
  > > Fabulous. I agree with @rkflx that before committing this, we should also prepare a similar patch for Dolphin to unify the behaviors.
  >
  >
  > That would be great, as this would solve bug 350932 <https://bugs.kde.org/show_bug.cgi?id=350932> & keep the behavior consistent across applications.
  
  
  In fact, perhaps we should consider this new automatically-disable-previews-at-small-sizes behavior to be a replacement for Dolphin's clunky independent-sliders-for-previews-and-non-previews feature. I suspect it was implemented that way to allay a similar concern (previews being useless at small sizes) but IMHO the solution in this patch is much more elegant and user-friendly

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D12321

To: anemeth, #vdg, #frameworks, ngraham, rkflx, #dolphin
Cc: xyquadrat, sharvey, rkflx, ngraham, #frameworks, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180419/6faf8982/attachment.htm>