D11022: Resize the "normal" folder icons

[Nathaniel Graham]

ngraham added a comment.


  In D11022#238047 <https://phabricator.kde.org/D11022#238047>, @elvisangelaccio wrote:
  
  > In D11022#235434 <https://phabricator.kde.org/D11022#235434>, @xyquadrat wrote:
  >
  > > In D11022#234894 <https://phabricator.kde.org/D11022#234894>, @elvisangelaccio wrote:
  > >
  > > > If we make all icons of the same size (which I'm not sure I'd like), that code would become useless and should then be removed.
  > >
  > >
  > >
  > >
  > > 1. I am totally fine with abandoning this patch, but it was brought up on a few occasions on Bugzilla so I simply went ahead & made a patch
  >
  >
  > The thing is, for one user that complains on bugzilla there are probably 10x people who are happy about the current implementation. Some of them *will* complain after this patch goes in.
  
  
  What do we suspect that they will complain about? The icons/previews being too big if they fully fill the preview space?
  
  To me, and to the version of me where I impersonate a Regular User™, the current implementation feels buggy:
  
  - Some folders and icons have a maximum size in the preview pane, but others don't (putting on my "normal user" hat, I would have no idea why this was, and it would seem like a bug)
  - Some folders that show a preview of their contents in the main view, but not in the information panel (not related to this patch, but still feels like a bug to Mr. or Mrs. Normal User)

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

To: xyquadrat, #dolphin, ngraham
Cc: elvisangelaccio, ngraham, broulik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180401/8365cf2b/attachment.htm>