Skip to content

Shorten the text in /css/css-view-transitions/new-content-is-inline.html to not linewrap #1252

Description

@dholbert

Test List

https://wpt.fyi/results/css/css-view-transitions/new-content-is-inline.html

Rationale

Prior to web-platform-tests/wpt#45809 , the text-strings in this WPT did not linewrap at all in Chrome or Safari -- so that demonstrates that linewrapping & multi-line text strings are not part of what it's trying to test.[1]

But when it was migrated to use 20px-sized Ahem (in web-platform-tests/wpt#45809 ), the text started linewrapping. That happens to trigger Firefox bug 2012343 (which is also independently and more-directly tested by another view transitions test, multiline-span-with-overflowing-text-and-box-decorations.html.

Hence, I propose that we adjust this test to shorten the text so that it doesn't linewrap anymore, in the same way that I recently did for the neighbor old-content-is-inline.html in #1250.

Benefits of this proposal:

  • Avoids triggering the unrelated Firefox bug (which as noted is already tested by a separate WPT) so that Firefox can pass this test.
  • Lets us remove the fuzzy annotation (I believe all browsers pass with no fuzz if we don't linewrap).
  • Makes this test much more consistent with the neighboring analogous test old-content-is-inline.html

[1] Moreover, if I modify a historical pre-Ahem version of the testcase slightly to make the text barely wrap in Chrome, it fails in Chrome; this isn't a Chrome bug, but is rather an indication that the test itself isn't robust in the face of linewrapping.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    test-change-proposalProposal to add or remove tests for an interop area

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions