From c763b1e9f99dd5c99b673b74d08919361f5b936d Mon Sep 17 00:00:00 2001 From: Jo Steven Novaryo Date: Fri, 9 Jan 2026 02:03:38 +0000 Subject: [PATCH 1/2] Add testdriver WPTs Signed-off-by: Jo Steven Novaryo --- ...oll-element-affected-by-parent-margin.html | 55 +++++++++++++++++++ 1 file changed, 55 insertions(+) create mode 100644 css/css-overflow/scroll-element-affected-by-parent-margin.html diff --git a/css/css-overflow/scroll-element-affected-by-parent-margin.html b/css/css-overflow/scroll-element-affected-by-parent-margin.html new file mode 100644 index 00000000000000..de65bb80014cdd --- /dev/null +++ b/css/css-overflow/scroll-element-affected-by-parent-margin.html @@ -0,0 +1,55 @@ + + +Should be able to scroll an element that is offset by the ancestor's margin + + + + + + + + + +
+
+
+
+ + From 81f07a3c9b3d7d591bbee3dd7aa617796f142326 Mon Sep 17 00:00:00 2001 From: Jo Steven Novaryo Date: Fri, 9 Jan 2026 02:03:38 +0000 Subject: [PATCH 2/2] Apply suggestions Signed-off-by: Jo Steven Novaryo --- ...overflow-auto-scrolling-with-margins.html} | 20 ++++++++----------- 1 file changed, 8 insertions(+), 12 deletions(-) rename css/css-overflow/{scroll-element-affected-by-parent-margin.html => overflow-auto-scrolling-with-margins.html} (64%) diff --git a/css/css-overflow/scroll-element-affected-by-parent-margin.html b/css/css-overflow/overflow-auto-scrolling-with-margins.html similarity index 64% rename from css/css-overflow/scroll-element-affected-by-parent-margin.html rename to css/css-overflow/overflow-auto-scrolling-with-margins.html index de65bb80014cdd..2acb543483c3be 100644 --- a/css/css-overflow/scroll-element-affected-by-parent-margin.html +++ b/css/css-overflow/overflow-auto-scrolling-with-margins.html @@ -1,6 +1,5 @@ - -Should be able to scroll an element that is offset by the ancestor's margin +Should be able to interactively scroll an overflow: auto element with a margin @@ -20,7 +19,7 @@ overflow: auto; scrollbar-width: none; } -#target { +#item { position: relative; top: 50px; left: 50px; @@ -30,7 +29,7 @@ }
-
+
@@ -38,18 +37,15 @@ promise_test(async (t) => { await waitForCompositorCommit(); let rect = scroller.getBoundingClientRect(); - const actions = new test_driver.Actions().scroll(rect.left+10, rect.top+10, 1000, 1000); + const actions = new test_driver.Actions().scroll(rect.left + 10, rect.top + 10, 1000, 1000); actions.send(); let scroll_delta_promise = new Promise((resolve) => { - let onscroll = (evt) => { - resolve([scroller.scrollLeft, scroller.scrollTop]); - } - scroller.addEventListener("scroll", onscroll); + scroller.addEventListener("scroll", () => resolve([scroller.scrollLeft, scroller.scrollTop])); }); let [horizontal_delta, vertical_delta] = await scroll_delta_promise; - assert_not_equals(horizontal_delta, 0, "Element horizontally scrolled"); - assert_not_equals(vertical_delta, 0, "Element vertically scrolled"); - }, "Element that is offset by the margin of it's ancestor should be able to be scrolled"); + assert_equals(horizontal_delta, 50, "Element scrolled horizontally"); + assert_equals(vertical_delta, 50, "Element scrolled vertically"); + }, "Margins should not interfere with interactive scrolling of a scroll container");