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)

enhancement

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.
>  
>
Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Component: CSS Parsing and Computation → web-platform-tests
Product: Core → Testing
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
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
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.