D29806: Thumbnail folders: scale down sub image when their height is too big

[Nathaniel Graham]

ngraham added a comment.


  Haha the "before" image is hilarious!

INLINE COMMENTS

> thumbnail.cpp:415
> +    } else
> +    if ((image.size().height() > imageTargetSize.height()) && (imageTargetSize.height() != 0)) {
> +        scaling = float(imageTargetSize.height()) / float(image.size().height());

Is this an `else if (...) {` or an `else { if (...) {`

REPOSITORY
  R320 KIO Extras

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

To: meven, #frameworks, ngraham
Cc: kde-frameworks-devel, kfm-devel, waitquietly, azyx, nikolaik, pberestov, iasensio, aprcela, fprice, LeGast00n, cblack, fbampaloukas, alexde, Codezela, feverfew, meven, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, rdieter, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200518/bb882361/attachment.htm>