Re: [csswg-drafts] [css-images] Should the values of image-orientation include the <angle> variants? (#4164)

The CSS Working Group just discussed `Should the values of image-orientation include the <angle> variants?`, and agreed to the following:

* `RESOLVED: Update note saying this is not for implementation and will be dropped`

<details><summary>The full IRC log of that discussion</summary>
&lt;dael> Topic: Should the values of image-orientation include the &lt;angle> variants?<br>
&lt;dael> github: https://github.com/w3c/csswg-drafts/issues/4164<br>
&lt;dael> smfr: WK is orkging on image-orientation. there's one with angles and one without. ANy other browsers interested in angle variants?<br>
&lt;dael> fantasai: B/c we made from-image default orientation I don't think strong use case for none. If not having web compat problems is this a necessary property? Still need definition b/c css print profile. If defaulting to exif do we nee dproperty at all?<br>
&lt;dbaron> I didn't think the &lt;angle> values were useful...<br>
&lt;dael> TabAtkins: I don't recall affermative use cases for none<br>
&lt;dael> myles: easier to add css to fix a busted image then to rotate<br>
&lt;dael> fantasai: True. Ideally information should be in image or html. If photo is sideways it's data is wrong<br>
&lt;dael> [everyone tlaks]<br>
&lt;emilio> q+<br>
&lt;Rossen_> q?<br>
&lt;dael> fantasai: If you turn off css or in reader mode you want it to be upright. If image is wrong you should fundamentally fix and not patch with css<br>
&lt;emilio> https://groups.google.com/d/msg/mozilla.dev.platform/A1aaENcsR6k/fPB1AvyaCAAJ<br>
&lt;dael> emilio: Gecko impl the angle values and unshipped when spec deprecated it. I don't think it's a lot of work to re-implement them<br>
&lt;dael> dbaron: I also don't think that useful and a weird use of angles<br>
&lt;dael> myles: Under spec b/c don't say which orientation rotated from<br>
&lt;fantasai> rotated from the orientation before applying EXIF<br>
&lt;dael> TabAtkins: q on myles comment. Use case was something where image pixel are correct and exif is busted? Or more broad?<br>
&lt;svoisen> jensimmons: It's Charlie's last week so, while it would be nice to have additional language for clarity on rendering of wavy lines, it's not something we will get to in the very near term.<br>
&lt;dael> myles: Yes. Comment not about nalge value, but presence of property<br>
&lt;fantasai> svoisen, tell Charlie to make it look good :)<br>
&lt;fantasai> svoisen, since that's basically what the spec is going to try to say ;)<br>
&lt;jensimmons> fantasai: that’s what I said…. just make it purtier<br>
&lt;fantasai> s/fantasai:/fantasai,/<br>
&lt;dael> TabAtkins: With regards to angles unspec implication was rotation from none...says 0deg corrisponds to none. Implies you rotate from that. I don't think underspec, but can be tweaked. Hopefully don't need to be impl. They're there b/c print renderers have them.<br>
&lt;myles> TabAtkins: where does it say that 0 means none?<br>
&lt;myles> i don't see it<br>
&lt;svoisen> fantasai: Ha, fair enough :) We will have to save that as a task for someone else.<br>
&lt;dael> smfr: fantasai suggested removing prop. But if Moz has been shipping with from-image removing is tricky. emilio do you have info on how long shipped?<br>
&lt;Rossen_> ack emilio<br>
&lt;dael> emilio: I don't think we have use counters. Could add. Been shipping for a long time if I recall. I'll find a link<br>
&lt;dael> dbaron: If we're going to try and change default I would suspect that any of the things using it are doing so to set from-image not none<br>
&lt;dael> fantasai: +1<br>
&lt;dael> fantasai: Unless using it to set a value other than from-image there's not a change if we remove property. Already resolved to change initial to from-image so might not need property<br>
&lt;dael> smfr: Possible to get photos with bad exif information. If you pic straight down you get an angle. I can imagine trying to fix that. It does fail with things that strip css<br>
&lt;dael> fantasai: My inclination is impl that support this try and switch to from-image as default. Impl that don't change to from or exif. If that works we try and remove property. If it doesn't we keep<br>
&lt;dbaron> What fantasai just said sounds like a good plan to me.<br>
&lt;dael> plinss: If building photo editing software might want to show image naturally as it is and then manually rotate<br>
&lt;dael> TabAtkins: Editing you're parsing photo into a canvas?<br>
&lt;dael> plinss: Maybe. Could parse exif data yourself, but there is utility to keeping. Agree from-image and none ar only prop with from-image as default<br>
&lt;dael> Rossen_: unshipping of moz behavior and resolve on that behavior<br>
&lt;dael> Rossen_: Which way are we leaning?<br>
&lt;dael> TabAtkins: Fine with dropping if impl are okay on supposition we can make switch to from-image<br>
&lt;dael> plinss: Compat risk when I was writing code for this you don't know if browser will rotate and can't tell. Anyone with code that cares about this is already screwed so wouldn't worry about compat.<br>
&lt;dael> plinss: WOuld be nice if you can tell browser rotated but don't know how to tell in css<br>
&lt;dael> fantasai: Query size of box if it's not square and get aspect ratio. Can tell you a bit<br>
&lt;dael> plinss: But then have to parse image and then you might as well parse exif data<br>
&lt;dael> Rossen_: Leaning toward dropping image-orientation<br>
&lt;dael> fantasai: Prop: update note in draft to indicate we might drop the entire property for browsers and keep a definition with the &lt;angle> values for historical reasons and say it's optional. Then remove. Or information this is in css print by not rec for impl<br>
&lt;dael> Rossen_: Prop: Update note saying this is not for implementation and will be dropped<br>
&lt;dael> Rossen_: Objections?<br>
&lt;dael> RESOLVED: Update note saying this is not for implementation and will be dropped<br>
</details>


-- 
GitHub Notification of comment by css-meeting-bot
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/4164#issuecomment-521325222 using your GitHub account

Received on Wednesday, 14 August 2019 16:45:27 UTC