-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Add test to check scrollable overflow of abspos elements #26657
Add test to check scrollable overflow of abspos elements #26657
Conversation
<div class="abspos" style="height: 500px; left: -50px;"></div> | ||
</div> | ||
<div class="containing-block" data-expected-scroll-width="100" data-expected-scroll-height="550"> | ||
<div class="abspos" style="height: 500px; left: -50px; top: 50px;"></div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: the order in height: 500px; left: -50px; top: 50px;
makes it harder to see what's happening since the horizontal left: -50px
is placed between the vertical declarations. IMO would be clearer as height: 500px; top: 50px; left: -50px;
. Ditto for the following cases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I had already merged it. I've fixed that in ##26659.
Apply suggestion from review after the test has landed: web-platform-tests#26657 (review)
Apply suggestion from review after the test has landed: #26657 (review)
…-001.html, a=testonly Automatic update from web-platform-tests Fix positon-absolute-scrollable-overflow-001.html Apply suggestion from review after the test has landed: web-platform-tests/wpt#26657 (review) -- Address review comment -- wpt-commits: a032b341c4dcd16ca4551da719a8c83a2e55d910, 819004e487f30f8f88c71f37c7d9f80f2aed0064 wpt-pr: 26659
…-001.html, a=testonly Automatic update from web-platform-tests Fix positon-absolute-scrollable-overflow-001.html Apply suggestion from review after the test has landed: web-platform-tests/wpt#26657 (review) -- Address review comment -- wpt-commits: a032b341c4dcd16ca4551da719a8c83a2e55d910, 819004e487f30f8f88c71f37c7d9f80f2aed0064 wpt-pr: 26659
No description provided.