overflow-padding.html doesn't work with overlay scrollbars (can fail on Android after bug 1700858)
Categories
(Core :: Layout: Scrolling and Overflow, defect)
Tracking
()
People
(Reporter: TYLin, Unassigned)
References
(Depends on 1 open bug)
Details
After my patch in bug 1527949, we still fail overflow-padding.html on Android.
TEST-UNEXPECTED-FAIL | /css/css-overflow/overflow-padding.html | Container padding is applied approriately to block/inline children. - assert_true: vertical scrollbar 1 expected true got false
Comment 1•4 years ago
|
||
Is this because of overlay scrollbars? If so, how does the test work on macOS?
Reporter | ||
Comment 2•4 years ago
|
||
[cc hiro]
Is this because of overlay scrollbars? If so, how does the test work on macOS?
I guess so. Maybe we setup our macOS on CI to have scrollbar always show? I run this test locally with my patches in bug 1527949. The test did fail when I switch to use overlay scrollbar, but pass when I set the scrollbar to always show.
The test uses the following helpers to detect the presence of the scrollbar assuming the scrollbar does take up some space. That doesn't seem to work with overlay scrollbar.
function hasHorizontalScrollbar(el) {
return (el.offsetHeight - el.clientHeight) > 0;
}
function hasVerticalScrollbar(el) {
return (el.offsetWidth - el.clientWidth) > 0;
}
Perhaps it is more reasonable to use scrollWidth
/scrollHeight
?
function hasHorizontalScrollbar(el) {
return (el.scrollWidth - el.offsetWidth) > 0;
}
function hasVerticalScrollbar(el) {
return (el.scrollHeight - el.offsetHeight) > 0;
}
Comment 3•4 years ago
|
||
Yeah, those helpers look a bit bogus in presence of overlay scrollbars....
Comment 4•4 years ago
|
||
(In reply to Ting-Yu Lin [:TYLin] (UTC-7) from comment #2)
[cc hiro]
Is this because of overlay scrollbars? If so, how does the test work on macOS?
I guess so. Maybe we setup our macOS on CI to have scrollbar always show?
Yes, I think so. A lot of tests still don't work with overlay.
Comment 5•4 years ago
|
||
Last time I checked (but it's a bit long time ago), Chromium didn't run any wpt tests on Android, so maybe the test also fails on mobile Chromium?
Reporter | ||
Updated•4 years ago
|
Reporter | ||
Comment 6•4 years ago
|
||
I changed the test in https://github.com/web-platform-tests/wpt/pull/28328, and it has been imported into gecko in bug 1702383.
Description
•