Rounding issue causes blurry image in PHP about jcrop HOT 4 CLOSED

Additional information - up to the width of 2000px, there are only 6 available selections for 740x656 that wouldn't cause a distortion in the resized PHP image:

925x820
1110x984
1295x1148
1480x1312
1665x1476
1850x1640

And if I changed that to 740x640, the amount of non-decimal choices would increase significantly:

777x672
814x704
851x736
888x768
925x800
962x832
999x864
1036x896
1073x928
1110x960
1147x992
1184x1024
1221x1056
1258x1088
1295x1120
1332x1152
1369x1184
1406x1216
1443x1248
1480x1280
1517x1312
1554x1344
1591x1376
1628x1408
1665x1440
1702x1472
1739x1504
1776x1536
1813x1568
1850x1600
1887x1632
1924x1664
1961x1696
1998x1728

from jcrop.

There are definitely precision issues when working with aspect ratios, resizing, etc. Just today I committed an update that removes all of the internal rounding that Jcrop was doing. The claim of the author is that this improves precision, which makes sense to me. If possible, can you retest with current master branch and advise if that improves on your issue? Other than that, there's not a lot of magic I can do to improve precision that isn't specific to a desired behavior that may vary from case to case.

from jcrop.

Tried the new version. I don't know what you changed, but IT WORKS! Can't believe it. Images are resized and the end result is high-quality. And the only thing I changed is I upgrade jcrop to latest version - didn't change any of the PHP. Thank you so much!

from jcrop.

Excellent! And thank you for letting me know. Some precision problems are inherent with all the scaling/division/etc... but in the code originally, much effort was made to be working with round numbers...I guess my thinking was that it would cause some obscure browser to choke on the CSS if it wasn't whole numbers. Anyway, this magnified the problem.

Someone recently submitted a patch that removed all the instances of rounding and claimed that it improved the precision. I had actually thought about this before (despite, if you can't tell, being lame at math) so I figured it was a logical change. Really great to hear that it helped a real life problem!

from jcrop.