Closed
Bug 1456739
Opened 6 years ago
Closed 5 years ago
[wpt-sync] Sync PR 10627 - [css-text] overflow-wrap and intrinsic sizing
Categories
(Testing :: web-platform-tests, enhancement, P4)
Testing
web-platform-tests
Tracking
(firefox67 fixed)
RESOLVED
FIXED
mozilla67
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 10627 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/w3c/web-platform-tests/pull/10627 Details from upstream follow. Florian Rivoal <git@florian.rivoal.net> wrote: > [css-text] overflow-wrap and intrinsic sizing > > Goes together with the spec changes for https://github.com/w3c/csswg-drafts/issues/2465 > > I am reasonably confident that the first 2 tests are supported by the spec. > > I am less sure about the 3rd and the 4th, altought I believe that they are supported by the intent of the spec. The spec says: > > > [...] must either hang the white space or visually collapse the character advance widths of any overflowing spaces such that they don’t take up space in the line. > > For browsers that chose the first alternative ("hang the white space"), then the 3rd and 4th test should be correct. > > For those that chose the second one ("visually collapse the character advance widths of any overflowing spaces such that they don’t take up space in the line"), I am less sure. Maybe they are correct, or maybe the intrinsic sizing happens first and makes room for the spaces, and then there's no need to collapse the advance width of overflowing spaces since no space overflows. > > Browsers that chose that option (Safari and Chrome) are not in full agreement: they both fail test 4, but Safari passes test 3. > >
Reporter | ||
Updated•6 years ago
|
Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core
Reporter | ||
Comment 1•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=85354e370a5008f4840318c9086240b0c14509f9
Reporter | ||
Comment 2•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=18c568280be59467183f7181f4b54a4c8265d18c
Reporter | ||
Updated•5 years ago
|
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Reporter | ||
Updated•5 years ago
|
Component: CSS Parsing and Computation → web-platform-tests
Product: Core → Testing
Reporter | ||
Updated•5 years ago
|
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Reporter | ||
Updated•5 years ago
|
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Reporter | ||
Updated•5 years ago
|
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Reporter | ||
Comment 3•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=44e9301ae0999651b7ffc026e676f8d052ea0344
Reporter | ||
Comment 4•5 years ago
|
||
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/548c17919547 [wpt PR 10627] - [css-text] white-space and intrinsic sizing, a=testonly
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/a83642423ce2 [wpt PR 10627] - [css-text] white-space and intrinsic sizing, a=testonly
Comment 7•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in
before you can comment on or make changes to this bug.
Description
•