Skip to content

Conversation

@msxf-matt
Copy link

bubble list should depend on event->CanBubble() instead of capture_list, which depends on event->CanCapture. Otherwise, the bubbling phase would be skipped when CanCapture() returns false.

bubble list should depend on `event->CanBubble()` instead of `capture_list`, which depends on `event->CanCapture`. Otherwise,  the bubbling phase would be skipped when CanCapture() returns false.
@msxf-matt msxf-matt requested a review from etkmao as a code owner December 22, 2025 08:47
@github-actions github-actions bot added dom: tree size: s Denotes a PR that changes 10-99 lines labels Dec 22, 2025
@hippy-service
Copy link

hippy-service bot commented Dec 22, 2025

Hi, @msxf-matt. Thanks for your PR! 👏

🏷️ You can leave a comment in this PR with #help tag when you need help (e.g. some status checks run failed due to internal issue), admin team members will help asap.

@tencent-adm
Copy link
Member

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@hippy-service
Copy link

hippy-service bot commented Dec 22, 2025

After a quick scan, I have approved workflow to run.

Details

🏷️ New commits in this PR would not be tested automatically until this pull request is reviewed by our collaborators.
🏷️ No need to worry about the status of merge_guard and [gh] pull request merge guard / merge_guard (pull_request_target) checks, once this pull request is met merge requirements, it will be automatically converted to successful status.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dom: tree size: s Denotes a PR that changes 10-99 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants