-
Notifications
You must be signed in to change notification settings - Fork 68
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support auto-rotation based on EXIF data #81
Comments
Seems a good idea. Sharp has a global useExifOrientation option. We had it in previous versions of IPX (See #13) which can add back support and possibly enable by default. However not sure what was reasons sharp kept it disabled by default. |
The global option does seem like a good idea |
Hi both, I started using ipx / nuxt-image and have just stumbled upon this issue for images in portrait format that I've been using. Indeed, it seems that the Any idea when this could be implemented in ipx? |
Turns out it works just by providing the For instance in my case in Nuxt with the image module, in
or for IPX only, with the Programatic API:
|
THat didn't seem to work for me. I'm using NextJS, does it make a difference? |
If sharp's
rotate()
function is called without arguments then it will try to automatically rotate the image based on exif data. ipx currently doesn't support this, and can only do explicit rotation. It would be good if it either allowedrotate
to be called without an angle, or if auto-rotation was the default (which would be my suggestion)The text was updated successfully, but these errors were encountered: