Skip to content
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

Issue with "image-orientation": cross-origin limitations #12484

Closed
matthewwithanm opened this issue Jan 28, 2022 · 2 comments · Fixed by #23513
Closed

Issue with "image-orientation": cross-origin limitations #12484

matthewwithanm opened this issue Jan 28, 2022 · 2 comments · Fixed by #23513
Labels
Content:CSS Cascading Style Sheets docs effort: small This task is a small effort. help wanted If you know something about this topic, we would love your help!

Comments

@matthewwithanm
Copy link

MDN URL: https://developer.mozilla.org/en-US/docs/Web/CSS/image-orientation

What information was incorrect, unhelpful, or incomplete?

As discussed in w3c/csswg-drafts/issues/5165, this behavior is no longer supported for cross-origin images. This severely limits its usefulness and means that the old default behavior (where EXIF data was ignored) can no longer be achieved for cross-origin images.

What did you expect to see?

The page should probably have a big callout about this.

Did you test this? If so, how?

Yes, I created a page that loads an image with an EXIF rotation and set image-orientation: none on it. When using a cross-origin image, the image was flipped according to the EXIF data (in Chrome). When using a same-origin image, the EXIF data was ignored.

MDN Content page report details
@github-actions github-actions bot added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Jan 28, 2022
@sideshowbarker sideshowbarker added Content:CSS Cascading Style Sheets docs help wanted If you know something about this topic, we would love your help! effort: small This task is a small effort. and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Jan 29, 2022
@tlylt
Copy link
Contributor

tlylt commented Jan 9, 2023

Hi there,

I am not sure if I can find any official reference on this change besides the csswg-drafts issue, @sideshowbarker should we still proceed now with adding a call-out on this or?

@sideshowbarker
Copy link
Member

should we still proceed now with adding a call-out on this or?

Yeah — MDN should document what browsers actually do, even in cases where it differs from what the current spec states.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:CSS Cascading Style Sheets docs effort: small This task is a small effort. help wanted If you know something about this topic, we would love your help!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants