Skip to content

[Editorial review] BiDi - Add pages for navigation and page load events of browsingContext module - #44571

Open
dipikabh wants to merge 14 commits into
mdn:mainfrom
dipikabh:bidi-browsingcontext-nav-events
Open

[Editorial review] BiDi - Add pages for navigation and page load events of browsingContext module#44571
dipikabh wants to merge 14 commits into
mdn:mainfrom
dipikabh:bidi-browsingcontext-nav-events

Conversation

@dipikabh

@dipikabh dipikabh commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Description

This PR adds pages for the following navigation and page load events:

  • browsingContext.navigationStarted
  • browsingContext.navigationCommitted
  • browsingContext.domContentLoaded
  • browsingContext.load
  • browsingContext.navigationFailed
  • browsingContext.fragmentNavigated
  • browsingContext.historyUpdated

Note

browsingContext.navigationAborted is not documented yet. The BCD key is also missing for this event. I'll cover it when there is an implementation.

Spec links

Related issue

Doc issue: mdn/mdn#851

@dipikabh
dipikabh requested review from a team as code owners June 25, 2026 00:32
@dipikabh
dipikabh requested review from hamishwillee and pepelsbey and removed request for a team June 25, 2026 00:32
@github-actions github-actions Bot added Content:WebDriver WebDriver docs size/l [PR only] 501-1000 LoC changed labels Jun 25, 2026
@dipikabh
dipikabh requested review from a team and removed request for hamishwillee and pepelsbey June 25, 2026 00:33
@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

@juliandescottes juliandescottes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

(answers for the inline questions, reviewing the rest of the PR still, thanks!)

@juliandescottes juliandescottes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, just some comments on my end, will take another look once they are addressed.

@dipikabh

Copy link
Copy Markdown
Contributor Author

Thanks a lot for the review, @juliandescottes!
I've finally been able to get around to addressing your comments.

Let me know what you think and if any more updates are needed. Thanks!

@dipikabh
dipikabh requested a review from juliandescottes July 14, 2026 01:32
@dipikabh

Copy link
Copy Markdown
Contributor Author

Thanks @juliandescottes for checking.

@pepelsbey, sending to you now for an editorial review.

@dipikabh
dipikabh requested a review from pepelsbey July 17, 2026 02:50
@dipikabh dipikabh changed the title [Tech review] BiDi - Add pages for navigation and page load events of browsingContext module [Editorial review] BiDi - Add pages for navigation and page load events of browsingContext module Jul 17, 2026

@juliandescottes juliandescottes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @dipikabh !

Some additional comments, but I think this is good to go!

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

Labels

Content:WebDriver WebDriver docs size/l [PR only] 501-1000 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants