Skip to content

devtools: Show attached event listeners in inspector tab#42355

Merged
simonwuelker merged 2 commits intoservo:mainfrom
simonwuelker:inspector-eventlistener
Feb 11, 2026
Merged

devtools: Show attached event listeners in inspector tab#42355
simonwuelker merged 2 commits intoservo:mainfrom
simonwuelker:inspector-eventlistener

Conversation

@simonwuelker
Copy link
Copy Markdown
Member

@simonwuelker simonwuelker commented Feb 5, 2026

This change makes the devtools inspector show any event listeners that are attached to a node. The primary motivation is making the devtools more useful for debugging real-world websites.

image

You can also click on the event listener to show some more info, though most of that (everything except the event type and the event phase) is currently just placeholder text:

image

Testing: This change adds a new test

@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Feb 5, 2026
@simonwuelker simonwuelker force-pushed the inspector-eventlistener branch 2 times, most recently from 1757552 to c8fea08 Compare February 5, 2026 08:02
Copy link
Copy Markdown
Member

@eerii eerii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@servo-highfive servo-highfive added S-needs-rebase There are merge conflict errors. and removed S-awaiting-review There is new code that needs to be reviewed. labels Feb 11, 2026
Signed-off-by: Simon Wülker <simon.wuelker@arcor.de>
Signed-off-by: Simon Wülker <simon.wuelker@arcor.de>
@simonwuelker simonwuelker force-pushed the inspector-eventlistener branch from c8fea08 to 28d126f Compare February 11, 2026 13:22
@servo-highfive servo-highfive added S-awaiting-review There is new code that needs to be reviewed. and removed S-needs-rebase There are merge conflict errors. labels Feb 11, 2026
@simonwuelker simonwuelker added this pull request to the merge queue Feb 11, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Feb 11, 2026
Merged via the queue into servo:main with commit 929e275 Feb 11, 2026
33 checks passed
@simonwuelker simonwuelker deleted the inspector-eventlistener branch February 11, 2026 15:49
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-devtools S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants