mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-08-16 07:20:46 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibWeb: Dispatch click events using the correct button/buttons

Browse source 
We were generating click events always using the primary mouse button
instead of the provided button, and with the buttons field set to that
provided button.
This commit is contained in:
Timothy Flynn 2024-09-29 08:54:29 -04:00 • committed by Andreas Kling
commit dd5a0361f2
Notes: github-actions[bot] 2024-09-29 17:31:17 +00:00
3 changed files with 36 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

4
Tests/LibWeb/Text/expected/UIEvents/mouse-events.txt
View file

@ -8,6 +8,10 @@ mouseout target.id=(inner) currentTarget.id=(inner), relatedTarget.id=(outer)
mouseout target.id=(inner) currentTarget.id=(outer), relatedTarget.id=(outer) mouseout target.id=(inner) currentTarget.id=(outer), relatedTarget.id=(outer)
mouseleave target.id=(inner) currentTarget.id=(inner), relatedTarget.id=(outer) mouseleave target.id=(inner) currentTarget.id=(inner), relatedTarget.id=(outer)
mouseover target.id=(outer) currentTarget.id=(outer), relatedTarget.id=(inner) mouseover target.id=(outer) currentTarget.id=(outer), relatedTarget.id=(inner)
> click #outer
click target.id=(outer) currentTarget.id=(outer), button=0
> auxclick #outer
auxclick target.id=(outer) currentTarget.id=(outer), button=1
> click document.body > click document.body
mouseout target.id=(outer) currentTarget.id=(outer), relatedTarget.id=(body) mouseout target.id=(outer) currentTarget.id=(outer), relatedTarget.id=(body)
mouseleave target.id=(outer) currentTarget.id=(outer), relatedTarget.id=(body) mouseleave target.id=(outer) currentTarget.id=(outer), relatedTarget.id=(body)

30
Tests/LibWeb/Text/input/UIEvents/mouse-events.html
View file

@ -34,17 +34,26 @@ function handleMouseEnter(e) {
function handleMouseLeave(e) { function handleMouseLeave(e) {
println(`mouseleave target.id=(${e.target.id}) currentTarget.id=(${e.currentTarget.id}), relatedTarget.id=(${e.relatedTarget.id})`); println(`mouseleave target.id=(${e.target.id}) currentTarget.id=(${e.currentTarget.id}), relatedTarget.id=(${e.relatedTarget.id})`);
} }
function handleClick(e) {
println(`click target.id=(${e.target.id}) currentTarget.id=(${e.currentTarget.id}), button=${e.button}`);
}
function handleAuxClick(e) {
println(`auxclick target.id=(${e.target.id}) currentTarget.id=(${e.currentTarget.id}), button=${e.button}`);
}
outer.onmouseover = handleMouseOver; outer.onmouseover = handleMouseOver;
outer.onmouseout = handleMouseOut; outer.onmouseout = handleMouseOut;
outer.onmouseenter = handleMouseEnter; outer.onmouseenter = handleMouseEnter;
outer.onmouseleave = handleMouseLeave; outer.onmouseleave = handleMouseLeave;
outer.onclick = handleClick;
outer.onauxclick = handleAuxClick;
inner.onmouseover = handleMouseOver; inner.onmouseover = handleMouseOver;
inner.onmouseout = handleMouseOut; inner.onmouseout = handleMouseOut;
inner.onmouseenter = handleMouseEnter; inner.onmouseenter = handleMouseEnter;
inner.onmouseleave = handleMouseLeave; inner.onmouseleave = handleMouseLeave;
inner.onclick = handleClick;
inner.onauxclick = handleAuxClick;
const clickOnBody = () => { const clickOnBody = () => {
return new Promise(resolve => { return new Promise(resolve => {
@ -53,6 +62,21 @@ const clickOnBody = () => {
}); });
} }
const clickOnOuterBox = (button) => {
return new Promise((resolve, reject) => {
if (button == 0) {
document.body.onclick = () => { resolve(); };
internals.click(80, 80);
} else if (button == 1) {
document.body.onauxclick = () => { resolve(); };
internals.middleClick(80, 80);
} else {
println(`Unimplemented button click: ${button}`);
reject();
}
});
};
asyncTest(async done => { asyncTest(async done => {
// First move the mouse outside #outer to populate the MouseEvent.relatedTarget property // First move the mouse outside #outer to populate the MouseEvent.relatedTarget property
internals.movePointerTo(150, 150); internals.movePointerTo(150, 150);
@ -60,6 +84,10 @@ asyncTest(async done => {
internals.movePointerTo(10, 10); internals.movePointerTo(10, 10);
println("> move pointer over #outer"); println("> move pointer over #outer");
internals.movePointerTo(60, 60); internals.movePointerTo(60, 60);
println("> click #outer");
await clickOnOuterBox(0);
println("> auxclick #outer");
await clickOnOuterBox(1);
println("> click document.body"); println("> click document.body");
await clickOnBody(); await clickOnBody();
done(); done();

6
Userland/Libraries/LibWeb/Page/EventHandler.cpp
View file

@ -297,13 +297,13 @@ EventResult EventHandler::handle_mouseup(CSSPixelPoint viewport_position, CSSPix
bool run_activation_behavior = false; bool run_activation_behavior = false;
if (node.ptr() == m_mousedown_target) { if (node.ptr() == m_mousedown_target) {
if (button == UIEvents::MouseButton::Primary) { if (button == UIEvents::MouseButton::Primary) {
run_activation_behavior = node->dispatch_event(UIEvents::MouseEvent::create_from_platform_event(node->realm(), UIEvents::EventNames::click, screen_position, page_offset, client_offset, offset, {}, 1, button, modifiers).release_value_but_fixme_should_propagate_errors()); run_activation_behavior = node->dispatch_event(UIEvents::MouseEvent::create_from_platform_event(node->realm(), UIEvents::EventNames::click, screen_position, page_offset, client_offset, offset, {}, button, buttons, modifiers).release_value_but_fixme_should_propagate_errors());
} else if (button == UIEvents::MouseButton::Middle) { } else if (button == UIEvents::MouseButton::Middle) {
run_activation_behavior = node->dispatch_event(UIEvents::MouseEvent::create_from_platform_event(node->realm(), UIEvents::EventNames::auxclick, screen_position, page_offset, client_offset, offset, {}, 1, button, modifiers).release_value_but_fixme_should_propagate_errors()); run_activation_behavior = node->dispatch_event(UIEvents::MouseEvent::create_from_platform_event(node->realm(), UIEvents::EventNames::auxclick, screen_position, page_offset, client_offset, offset, {}, button, buttons, modifiers).release_value_but_fixme_should_propagate_errors());
} else if (button == UIEvents::MouseButton::Secondary) { } else if (button == UIEvents::MouseButton::Secondary) {
// Allow the user to bypass custom context menus by holding shift, like Firefox. // Allow the user to bypass custom context menus by holding shift, like Firefox.
if ((modifiers & UIEvents::Mod_Shift) == 0) if ((modifiers & UIEvents::Mod_Shift) == 0)
run_activation_behavior = node->dispatch_event(UIEvents::MouseEvent::create_from_platform_event(node->realm(), UIEvents::EventNames::contextmenu, screen_position, page_offset, client_offset, offset, {}, 1, button, modifiers).release_value_but_fixme_should_propagate_errors()); run_activation_behavior = node->dispatch_event(UIEvents::MouseEvent::create_from_platform_event(node->realm(), UIEvents::EventNames::contextmenu, screen_position, page_offset, client_offset, offset, {}, button, buttons, modifiers).release_value_but_fixme_should_propagate_errors());
else else
run_activation_behavior = true; run_activation_behavior = true;
} }