LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-12-25 20:22:18 +03:00
Code Issues Packages Projects Releases Wiki Activity

LibWeb: Skip page scrolling for wheel events consumed by scrollable box

Browse Source 
Fixes the bug when we scroll both scrollable box and page.
This commit is contained in:
Aliaksandr Kalenik 2024-02-29 10:35:25 +01:00 committed by Andreas Kling
parent 30d8c5f301
commit 90879a07ba
Notes: sideshowbarker 2024-07-17 00:16:31 +09:00 
Author: https://github.com/kalenikaliaksandr
Commit: https://github.com/SerenityOS/serenity/commit/90879a07ba
Pull-request: https://github.com/SerenityOS/serenity/pull/23402
3 changed files with 37 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Tests/LibWeb/Text/expected/wheel-events-consumed-by-scrollable-should-not-be-propagated-to-body.txt Normal file
View File

@ -0,0 +1,2 @@
scrollable.scrollTop: 100
window.scrollY: 0

32
Tests/LibWeb/Text/input/wheel-events-consumed-by-scrollable-should-not-be-propagated-to-body.html Normal file
View File

@ -0,0 +1,32 @@
<!DOCTYPE html>
<link rel="match" href="reference/scroll-using-mousewheel-event-ref.html" />
<style>
body {
border: 1px solid black;
margin: 0;
}
#scrollable {
width: 100px;
height: 100px;
overflow: scroll;
border: 1px solid black;
margin-bottom: 2000px;
}
#content {
width: 200px;
height: 200px;
background-color: magenta;
}
</style>
<script src="include.js"></script>
<div id="scrollable"><div id="content"></div></div>
<script>
test(() => {
internals.wheel(10, 10, 0, 1000);
const scrollable = document.getElementById("scrollable");
println("scrollable.scrollTop: " + scrollable.scrollTop);
println("window.scrollY: " + window.scrollY);
});
</script>

7
Userland/Libraries/LibWeb/Page/EventHandler.cpp
View File

@ -174,10 +174,9 @@ bool EventHandler::handle_mousewheel(CSSPixelPoint position, CSSPixelPoint scree
if (paintable) {
auto* containing_block = paintable->containing_block();
while (containing_block) {
if (containing_block->is_user_scrollable()) {
const_cast<Painting::PaintableBox*>(containing_block->paintable_box())->handle_mousewheel({}, position, buttons, modifiers, wheel_delta_x, wheel_delta_y);
break;
}
auto handled_scroll_event = const_cast<Painting::PaintableBox*>(containing_block->paintable_box())->handle_mousewheel({}, position, buttons, modifiers, wheel_delta_x, wheel_delta_y);
if (handled_scroll_event)
return true;
containing_block = containing_block->containing_block();
}