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

[css-images-3] image-orientation: none applies to non-decorative images too. #5245 #5294

Closed
wants to merge 1 commit into from

Conversation

heycam
Copy link
Contributor

@heycam heycam commented Jul 7, 2020

Fixes #5245.

@heycam heycam requested a review from fantasai July 7, 2020 03:56
@heycam
Copy link
Contributor Author

heycam commented Jul 7, 2020

I'm adding tests in https://phabricator.services.mozilla.com/D82471.

@frivoal frivoal added the css-images-3 Current Work label Jul 14, 2020
fantasai added a commit that referenced this pull request Oct 5, 2020
@fantasai
Copy link
Collaborator

fantasai commented Oct 5, 2020

Sorry for taking so long on this; there were a number of adjustments I knew I wanted to make so I procrastinated a lot on the review. But thanks for making sure the spec covers a wide range of the implications of the WG resolution!

Pushed some edits to the spec, which are better for having your PR to work from. :) I think we still have a bit of adjustment to do to clearly describe e.g. the SVG cases, so I'm going to leave this PR open until we're 100% sure we covered everything you have here.

fantasai added a commit that referenced this pull request Oct 5, 2020
…tive images as well as content images. Part II (because forgot to hit save or something) #5245 #5294
@fantasai
Copy link
Collaborator

OK. I think I've got everything covered now @heycam. Feel free to review and complain if not. :) In the meantime, I'm going to close this PR.

@fantasai fantasai closed this Nov 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css-images-3 Current Work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[css-images-3] change which images image-orientation applies to
3 participants