[go: up one dir, main page]

Skip to content
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

[css2][css-fonts] Tests to go with https://github.com/w3c/csswg-drafts/pull/1993 #8448

Merged
merged 5 commits into from
Dec 1, 2017

Conversation

frivoal
Copy link
Contributor
@frivoal frivoal commented Nov 27, 2017

w3c/csswg-drafts#1993 fixed / improved some definitions of line-height, the height of the content area of inline boxes, and of first-available font.

These are the tests to go with that spec PR.

These match with the spec changes in w3c/csswg-drafts#1993
regarding the line-height property.

The odd numbering starting at 201 rather that 001 is to avoid clashing
with existing similarly named tests.
These match with the spec changes in w3c/csswg-drafts#1993
regarding the height of the content area of inline boxes.
These test whether the the ex and ch units correctly depend on the first
font **that contains U+0020**, rather than the first font at all.

This matches the resolution from
w3c/csswg-drafts#1765,
and the spec pull request in w3c/csswg-drafts#1993
These test whether the line-height calculations define in CSS2.1
correctly depend on the first font **that contains U+0020**,
rather than the first font at all.

This matches the resolution from
w3c/csswg-drafts#1798,
and the spec pull request in w3c/csswg-drafts#1993
This tests whether the inline box content height calculations defined in
CSS2.1 correctly depend on the first font **that contains U+0020**,
rather than the first font at all.

This matches the resolution from
w3c/csswg-drafts#1798,
and the spec pull request in w3c/csswg-drafts#1993
@ghost
Copy link
ghost commented Nov 27, 2017

Build PASSED

Started: 2017-11-27 10:29:41
Finished: 2017-11-27 10:38:15

Failing Jobs

  • MicrosoftEdge:14.14393

Unstable Results

Browser: "Microsoftedge 14.14393" (failures allowed)

View in: WPT PR Status | TravisCI

Test Subtest Results Messages
/css/CSS2/visudet/content-height-001.html   PASS: 9
ERROR: 1
/css/CSS2/visudet/line-height-204.html   PASS: 2
FAIL: 8

Copy link
Contributor
@svgeesus svgeesus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These "first font with 0x20" tests seems straightforward and match the CSS WG resolution.

@svgeesus svgeesus merged commit e164bbf into web-platform-tests:master Dec 1, 2017
@frivoal
Copy link
Contributor Author
frivoal commented Dec 4, 2017

Thanks. I expected the merge for the tests and the PR to happen at roughly the same time. Now that the tests are merged it, we should merge in the spec part as well before long. Can you help me nudge the editor (Bert) and the other people who should review that?

@frivoal frivoal deleted the line-height branch March 21, 2019 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants