Skip to content

Commit

Permalink
Make tests for transforms on backgrounds of the root element not test…
Browse files Browse the repository at this point in the history
… scrollable overflow (#42493)

These tests are intended to test the effect of transform on the root
element on backgrounds that are propagated to the canvas.  They
coincidentally also test the effect of a transformed root element on the
scrollable overflow that applies to the viewport's scrollbars.

The scrollbar issue is underspecified as described in
w3c/csswg-drafts#9458, and implementations differ as to how a
transformed root element affects scrollbar extents on the viewport.

Because of this, this change replaces the previous adjustment of these
tests in #36476 (from matching some
implementations to matching others) with a change to the tests so that
they no longer test the extent of scrollbars on the root element.
  • Loading branch information
dbaron authored Oct 12, 2023
1 parent 6e6d5b9 commit 1e1d405
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,6 @@
<style>
html {
background: green;
/* Match scrollbar change caused by translateY transformation */
margin-top: -250vh;
}
</style>
<div style="height: 400vh;"></div>
</html>
1 change: 1 addition & 0 deletions css/css-transforms/transform-translate-background-001.html
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
html {
background: linear-gradient(to bottom, green 0%, green 50%, red 50%, red 100%);
transform: translate(0, -250vh);
overflow: hidden;
}
</style>
<div style="height: 400vh;"></div>
Expand Down
1 change: 1 addition & 0 deletions css/css-transforms/transform-translate-background-002.html
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
<style>
html {
background: linear-gradient(to bottom, green 0%, green 50%, red 50%, red 100%);
overflow: hidden;
}
</style>
<div style="height: 400vh;"></div>
Expand Down

0 comments on commit 1e1d405

Please sign in to comment.