D12321: Hide file preview when icon is too small

[Mark Gaiser]

markg added a comment.


  In D12321#252369 <https://phabricator.kde.org/D12321#252369>, @rkflx wrote:
  
  > In D12321#252337 <https://phabricator.kde.org/D12321#252337>, @markg wrote:
  >
  > > In those cases where you just browse through a gazillion icons (nothing with an icon picker or selecting icons, i didn't say any of that) becomes impossible in your future patch.
  > >  This patch makes it slightly more "inconvenient" to browse folders like that.
  >
  >
  > What "future patch" are you referring to, though?
  >
  > As far as I've understood, we don't plan to prevent people from viewing thumbnails for tiny icons anywhere (as long as they select a reasonable/medium grid size).
  >
  > In fact, with D12306: Improve grid icon layout in filepicker dialog <https://phabricator.kde.org/D12306> there probably won't be much of a difference between slider position with regard to grid spacing anyway, as "this patch improves the grid spacing in icons-on-top mode by making it looser for small icons", to give more room for the filename label.
  
  
  You said:
  
  > This automatic logic is already there. You are simply objecting to not allowing previews for small icons. Yes, we will remove this possibility which was there before.
  
  Which i interpreted as "previews for small images/icons won't be possible anymore in the future".
  Did i interpret that wrong?

REPOSITORY
  R241 KIO

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

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