Re: [csswg-drafts] [css-images-3] clarify which images image-orientation applies to (#5245)

The CSS Working Group just discussed `[css-images-3] clarify which images image-orientation applies to`, and agreed to the following:

* `RESOLVED: Make from-image's none value apply to background and any other images`

<details><summary>The full IRC log of that discussion</summary>
&lt;dael> Topic: [css-images-3] clarify which images image-orientation applies to<br>
&lt;dael> github: https://github.com/w3c/csswg-drafts/issues/5245<br>
&lt;dael> heycam: We have image orientation property that allows us turn off auto-re-orientation.<br>
&lt;dael> heycam: Some discussion about widening to all images a few months ago. Resolution from there was orientation meta data should be honored for decorative images. Makes sense.<br>
&lt;dael> heycam: Wasn't discussed if properties should be extended. As MIke noticed we have contradictory WPTs. Should image-orientation apply to decorative images?<br>
&lt;dino> q+<br>
&lt;dael> heycam: My feeling is it should b/c real purpose of property is let authors opt-out and it doesn't make sense to limit that to a subset of images<br>
&lt;Rossen_> ack dino<br>
&lt;dael> dino: Question is what would you do if want decorative to do one thing and non-decorative do another<br>
&lt;dael> heycam: You would need to introduce some other way of override orientation, maybe with image value syntax, or correct images<br>
&lt;dael> dino: Sort of leading question. In many cases will have to fix image or change content. Is it worth having this apply everywhere? I don't know<br>
&lt;dael> heycam: From experience of bug reports they were all content images. jpegs with image data rotated but tool didn't update meta data so new image-orientation made them wrong. Didn't get cases on decorative<br>
&lt;dael> fantasai: We had this discussion before and I believe resolution was only to apply to content. Idea was if we needed to apply to other we would introduce image notation syntax. This discussion was before auto xif but that was original discussion and conclusion<br>
&lt;dael> Rossen_: Is that 4165?<br>
&lt;dael> fantasai: older<br>
&lt;dael> fantasai: Let me change DoC<br>
&lt;smfr> s/change/see if it’s in the/<br>
&lt;dael> s/xif/EXIF<br>
&lt;fantasai> https://lists.w3.org/Archives/Public/www-style/2012Mar/0412.html<br>
&lt;dael> Rossen_: Where does this leave the current issue?<br>
&lt;dael> fantasai: Here's the issue. #50 in DoC. Conclusion was repalced elemtns and generated content but not background image<br>
&lt;dael> fantasai: Or any other images in CSS<br>
&lt;dael> fantasai: Images which are considered part of content are effected but no other<br>
&lt;dael> faceless2_: If purpose is undo exif makes sense to apply, but it's not weird to not. As long as it's documented.<br>
&lt;fantasai> "It applies only to content images (e.g. replaced elements and generated content), not decorative images (such as background-image)."<br>
&lt;dael> fantasai: Spec says applies to content not decorative.<br>
&lt;fantasai> https://drafts.csswg.org/css-images-3/#the-image-orientation<br>
&lt;dael> fantasai: Anything that's a replaced element in css model it applies to that. Everything else is not effected.<br>
&lt;chrishtr> q+<br>
&lt;Rossen_> ack fantasai<br>
&lt;Zakim> fantasai, you wanted to discuss history<br>
&lt;dael> heycam: Seems like notion of this property has changed over time. Original I want to set an orientation on an image. Now proprty is forI I have a problem caused by change in browsers and this gets me the old way back so I don't have to fix images I'm serving. In that PoV makes sense to apply as broadly as possible. Let's author set one property which goes to all images<br>
&lt;fantasai> This was issue 50 in the 2012 LC Disposition of Comments https://drafts.csswg.org/css-images-3/issues-lc-2012<br>
&lt;dael> chrishtr: I agree with heycam reasoning and that's what Chrome impl. There were cases where it was useful behavior for site compat<br>
&lt;smfr> webkit agrees with cameron too<br>
&lt;dael> fantasai: If we make this change it should only affect none value and others shouldn't apply to decorative<br>
&lt;Rossen_> ack chrishtr<br>
&lt;dael> heycam: I think we have resolution to remove other values.<br>
&lt;dael> fantasai: It's deprecated but has been impl and shipped in multiple impl, jsut nto browsers. We can remove from next spec level, but there needs to be a spec with those.<br>
&lt;dael> chrishtr: Who impl?<br>
&lt;dael> fantasai: Some printer based technology sihpped with onboard layout in printer chipset.<br>
&lt;dael> chrishtr: Okay so we say rotate ones won't apply to style images<br>
&lt;dael> fantasai: And support for that is marked as optional so browsers don't need to impl. But there needs to be spec for it<br>
&lt;dael> Rossen_: Any changes to L3 images?<br>
&lt;dael> fantasai: Yes if we want to make from-image and none value apply to background and other images we need to change spec to say it's all images associated with element<br>
&lt;dael> Rossen_: Is this going to be still valid for printer or are they okay b/c L2?<br>
&lt;dael> fantasai: They don't impl from-image I believe<br>
&lt;dael> Rossen_: Sorry, confused by statement that there were impl<br>
&lt;dael> fantasai: If we make change for other values than yes. THat's why I'm saying it shouldn't apply to the other values<br>
&lt;dael> Rossen_: Would that solution work for everybody<br>
&lt;dael> heycam: sgtm<br>
&lt;dael> Rossen_: Prop: Make from-image and none value apply to background and any other images?<br>
&lt;dael> fantasai: angle only applies to content<br>
&lt;dael> Rossen_: none value on from-image<br>
&lt;dael> Rossen_: Additional comments?<br>
&lt;dael> smfr: Can we resolve on cursor behavior and linked meta that are in GH issue?<br>
&lt;dael> Rossen_: Would this be the exception on cursor?<br>
&lt;dael> smfr: Makes sense for cursor to have same. Link and meta I don't know since usually don't apply CSS to them<br>
&lt;dael> fantasai: Not sure what a link or meta element would do<br>
&lt;dael> heycam: Some cases about favicons and similar images<br>
&lt;dael> smfr: First comment in issues has list of interesting items like embed object. We need to be explicit which of these we include<br>
&lt;dael> chrishtr: Canvas is an imparative API<br>
&lt;dael> smfr: Not sure abotu canvas I think right is API to expose exif to JS or extend draw image API<br>
&lt;tantek> presumably you can put images on link or meta via background-image or ::before and content property etc.<br>
&lt;dael> fantasai: Rest it should apply, every image associated to the element is effected as far as CSS is concerned. If it's a replaced element it's applied. Decorative elements would include background iamge and cursor<br>
&lt;dael> smfr: Source graphic in SVG<br>
&lt;dael> fantasai: Replaced element, isn't it?<br>
&lt;dael> smfr: I think it's paint source or whatveer it's called<br>
&lt;dael> heycam: I think that's rendered content of some subelement on dom tree<br>
&lt;dael> fantasai: Do you do image orientation of source, using, or neither element<br>
&lt;dael> heycam: Similar to canvas-draw-image b/c can reference another image element. Have minor incompat on where to look already. I think those cases can be handled in specs that define what it's referencing<br>
&lt;dael> smfr: Agree. We can resolve on what we discussed and say SVG may need refining<br>
&lt;dael> heycam: I'm happy to make cursor effected<br>
&lt;dael> Rossen_: Should we try to resolve on this and heycam or someone can open an issue on SVG to makes sure there's no additional items to associate to this decision?<br>
&lt;dael> smfr: sgtm<br>
&lt;smfr> s/smfr/heycam/<br>
&lt;dael> Rossen_: Still previous resolution. Objections?<br>
&lt;dael> RESOLVED: Make from-image's none value apply to background and any other images<br>
</details>


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

Received on Wednesday, 1 July 2020 23:23:15 UTC