-
Notifications
You must be signed in to change notification settings - Fork 711
[css21][css-inline] Height of an inline-level box with line-height:normal using primary and fallback fonts #1802
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
Comments
This means that the following description of the
Instead, the 1st and 2nd paragraphs of 10.8.1 Leading and half-leading, plus a sentence like the following, would be a good description of the
|
The following sentence in 2.1 is also wrong and should be removed, since for
|
The I had the hardest time boiling Here's the relevant snippet from the plugin: if (element_line_height === 'normal') {
var element_font_size = parseInt(computed_style.getPropertyValue('font-size'));
element_line_height = element_font_size * 1.125;
} else { Hopefully we can lock down what |
@tomhodgins Yes, inconsistencies in line height measurements are a pain. Attempting to lock down what If you find that any of the tests that I wrote is wrong or overlooking something, or that the conclusions I drew from them are problematic somehow, feedback would be greatly appreciated. |
One issue here is that the |
(Oh, except in Gecko -- that's the really weird behavior I was trying to remember -- considers the external leading metric only from the first font, and not from other fonts that happen to be used in the text!) |
This is part of the #1796 series. Read that issue for context
All browsers (Safari, Chrome, FF, Edge) agree, that the height of an inline-box with
line-height:normal
and which uses characters from the primary and fallback fonts goes from the top of the highest to the bottom of the lowest. In other words, the result is the same as the height of a line box containing one inline box for each of the font used.Test source
Ref source
Live test
Live ref
Given that we have interop, we should specify that.
The text was updated successfully, but these errors were encountered: