public/browser
1
0
Fork 0
mirror of https://github.com/lightpanda-io/browser.git synced 2026-03-22 04:34:44 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

Don't bubble "load" event to Window

Browse Source 
As per spec:

For legacy reasons, load events for resources inside the document (e.g., images)
do not include the Window in the propagation path in HTML implementations.
This commit is contained in:
Karl Seguin
2026-02-23 14:04:26 +08:00
parent 700a3e6ed9
commit 955351b5bd
1 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/browser/EventManager.zig
View File

@@ -309,11 +309,14 @@ fn dispatchNode(self: *EventManager, target: *Node, event: *Event, was_handled:
node = n._parent; node = n._parent;
} }
// Even though the window isn't part of the DOM, events always propagate // Even though the window isn't part of the DOM, most events propagate
// through it in the capture phase (unless we stopped at a shadow boundary) // through it in the capture phase (unless we stopped at a shadow boundary)
if (path_len < path_buffer.len) { // The only explicit exception is "load"
path_buffer[path_len] = page.window.asEventTarget(); if (event._type_string.eql(comptime .wrap("load")) == false) {
path_len += 1; if (path_len < path_buffer.len) {
path_buffer[path_len] = page.window.asEventTarget();
path_len += 1;
}
} }
const path = path_buffer[0..path_len]; const path = path_buffer[0..path_len];