Re: [whatwg/dom] Add pop-up light dismiss logic to event dispatching (PR #1117)

@josepharhar commented on this pull request.



> +     <li><p>Run {light dismiss open pop-ups} given <var>struct</var>'s <a
+     for=Event/path>invocation target</a>.

> This also seems like a layering violation.

Is HTML supposed to refer to DOM but not the other way around? I noticed that this spec refers to HTMLSlotElement from the HTML spec...

> Is this really supposed to happen for all events or just certain UI events?

Yes, it really happens for all events, at least in the [chromium implementation](https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/dom/node.cc;l=2885;drc=b5666c58a9cf7539e6f57c3e6e957c8fd956c650).

> (If you rebase and force push the CI error will go away.)

done

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/dom/pull/1117#discussion_r998591659
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/dom/pull/1117/review/1146344964@github.com>

Received on Tuesday, 18 October 2022 18:45:03 UTC