Re: [csswg-drafts] [css-inline-3] line-height and content height of inline box (#3820)

> it does not actually specify what the content height is

That text hasn't been ported from CSS2.1 yet, which is why it's incorporated by reference.

> Furthermore, ยง3.1 does specify the content height, but none of its definitions are affected by line-height.

It does seem that the interactions of `line-sizing` and `inline-sizing` weren't correctly handled.

> Furthermore, note that only negative half-leading is involved here, and it seems strange to specify that a reduced line-height should cause borders to overlap the ascenders and/or descenders of the text (at least in absence of padding). I suggest (i) removing the quoted parts that suggest that 'line-height' affects content height; and (ii) that either the contribution of an inline box to the line-box height is determined solely by one of margin-box height and line-height (or conceivably the minimum of the two); 

It's intentional that negative line-heights affect the content box of an inline, because if they didn't any child element inside a paragraph with e.g. `line-height: 0.9` would force the line-height back up to 1.

-- 
GitHub Notification of comment by fantasai
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/3820#issuecomment-482745533 using your GitHub account

Received on Friday, 12 April 2019 22:47:12 UTC