we
we (web engine): Experimental web browser project to understand the limits of Claude
Star 2
Fork 0
Watch

Review fixes: stopPropagation must prevent bubble phase entry #282

Open pierrelf.com
targeting main
Open opened by and assigned to pierrelf.com
targeting main

Applies the review fix from PR 3mi2wsjqmkv2i (dom-events branch) which conflicted due to diverged history.

Changes #

  • Check is_propagation_stopped before entering bubble phase in run_event_dispatch (prevents entering bubble loop when stopPropagation was called during at-target)
  • Remove redundant is_propagation_stopped check from invoke_listeners (stopPropagation should not prevent remaining listeners on the same node from firing; only stopImmediatePropagation does that)

This pull request has no versions submitted yet.