Skip to content

Conversation

@noamr
Copy link
Collaborator

@noamr noamr commented Oct 27, 2025

Copy link

@jnjaeschke jnjaeschke 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, thanks! (I have rights to approve now, but I'm not sure if my review is sufficient?)!

@noamr
Copy link
Collaborator Author

noamr commented Oct 28, 2025

@zcorpan PTAL

@zcorpan
Copy link
Member

zcorpan commented Oct 28, 2025

Is this covered by an existing test?

@noamr
Copy link
Collaborator Author

noamr commented Oct 28, 2025

Is this covered by an existing test?

Yes, the precommit handler tests would not work without this.

@zcorpan zcorpan merged commit a380924 into whatwg:main Oct 31, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Navigation API: Interception state is set incorrectly in #commit-a-navigate-event

3 participants