Skip to content

Check orientation of profile image

What does this MR do?

Fix those cases when the user chooses a image for his/her profile and that image has EXIF orientation information that usually come from cameras. It happened here #31187 (moved) with @bholtcamp. At first, I thought I would need to read the EXIF data and rotate the image by myself before sending it to cropper.js, but then I found out this was reported as a bug there since 2014, so they did something to help users, as you can read in README.

Are there points in the code the reviewer needs to double check?

It is a simple change that just configure cropper.js, but I'm a little worried with rotatable: true.

Why was this MR needed?

Depending on the camera you have, if you ever take a picture in portrait mode, the image ends up rotated with an EXIF information about its orientation, so image readers can use this data to draw the image correctly.

Screenshots (if relevant)

Before: wrong

After: correct

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

#31187 (moved)

Edited by username-removed-1616781

Merge request reports