Prevent image.js from computing "undefined" aspect ratio in rare edge case

theonion / betty-cropper

A work-in-progress image cropper.

MIT License

27 stars 7 forks source link

Prevent image.js from computing "undefined" aspect ratio in rare edge case #76

Closed mparent61 closed 8 years ago

mparent61 commented 8 years ago

@kand @MichaelButkovic @benghaziboy

Tightens up images.js aspect ratio computation.

Potential fix for production spike in 404s across all sites. I can't reproduce locally but it appears to be related to recirc sidebar images via image.js.

camsom commented 8 years ago

👍

MichaelButkovic commented 8 years ago

👍

mparent61 commented 8 years ago

Betty Cropper: High number of "undefined" aspect ratio 404s

kand commented 8 years ago

👍