Re: [csswg-drafts] [css-inline-3] line-height < 1 in a margin-box line layout model (#5178)

The CSS Working Group just discussed `[css-inline-3] line-height < 1 in a margin-box line layout model`.

<details><summary>The full IRC log of that discussion</summary>
&lt;Rossen_> Topic:  [css-inline-3] line-height &lt; 1 in a margin-box line layout model<br>
&lt;Rossen_> github: https://github.com/w3c/csswg-drafts/issues/5178<br>
&lt;dauwhe> fantasai: when we apply line height with positive half leading that's good<br>
&lt;dauwhe> ... if we have negative half-leading<br>
&lt;dauwhe> ... the size is less than the height of the font<br>
&lt;dauwhe> ... line-height: .8<br>
&lt;dauwhe> ... the ascent and descent of the font--you have to slice off the bottom part and take this as layout bounds<br>
&lt;dauwhe> ... in the new model we're not applying line-height to inline boxes<br>
&lt;dauwhe> ... this one doesn't get half leading, so it gets taller<br>
&lt;dauwhe> ... but we still have to reduce the size of inline boxes when there's negative half leading<br>
&lt;dauwhe> ... the proposal says we have to apply negative half leading... do we reduce the size of content box or margin box?<br>
&lt;dauwhe> ... in the proposal we say margin box<br>
&lt;dauwhe> ... what do people think about that?<br>
&lt;dauwhe> Rossen_: is there a reason for it to be other than the margin box?<br>
&lt;dauwhe> ... I think that makes sense<br>
&lt;dauwhe> florian: yes, it makes sense<br>
&lt;fantasai> myles, the diagram is similar to slide 52 in the slide deck, fwiw<br>
&lt;dauwhe> ... but if we would affect painting, it would be weird<br>
&lt;dauwhe> ... so it should be margin box<br>
&lt;dauwhe> Rossen_: any other opinions? If not, we can resolve.<br>
&lt;dauwhe> ... any objections?<br>
&lt;dauwhe> dbaron: not an objection<br>
&lt;dauwhe> ... the idea is fine<br>
&lt;dauwhe> ... what do numbers multiply by? Are they still what they used to mulitpyly by?<br>
&lt;dauwhe> fantasai: that's a good question<br>
&lt;dauwhe> myles: what's the argument to change?<br>
&lt;dauwhe> fantasai: if you want fixed you can set fixed<br>
&lt;dauwhe> myles: can you use px?<br>
&lt;dauwhe> florian: yes<br>
&lt;dauwhe> dbaron: but it's weird to do that because it inherits<br>
&lt;dauwhe> fantasai: I would say we treat them the same as currently<br>
&lt;dauwhe> ... youre trying to reduce size of line boxes by 205<br>
&lt;dauwhe> s/205/20%/<br>
&lt;dauwhe> ... you want ascent/descent to be shrunk a bit, to get the affect of setting solid<br>
&lt;dauwhe> dbaron: is the condition less than one, or what would make it less than the margin box<br>
&lt;dauwhe> fantasai: A+D doesn't necessarily add up to one em<br>
&lt;dauwhe> myles: that works even if line height is PX<br>
&lt;dauwhe> fantasai: yes<br>
&lt;dauwhe> Rossen_: are we ready to resolve?<br>
&lt;dauwhe> fantasai: I think so<br>
&lt;dauwhe> ... any objections?<br>
&lt;dauwhe> Resolved: apply negative half leading to margin box edge<br>
</details>


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

Received on Wednesday, 17 June 2020 17:17:48 UTC