Skip to content

[WebDriver Classic] Add pages for Back, Forward, Refresh, Get Title, and New Session commands#43109

Merged
dipikabh merged 14 commits intomdn:mainfrom
dipikabh:wd-commands-nav
Feb 27, 2026
Merged

[WebDriver Classic] Add pages for Back, Forward, Refresh, Get Title, and New Session commands#43109
dipikabh merged 14 commits intomdn:mainfrom
dipikabh:wd-commands-nav

Conversation

@dipikabh
Copy link
Contributor

Description

This work is part of the "Fill the gaps" project.

This PR adds documentation for Back, Forward, Refresh, and Get Title commands.

Note: While there is an ongoing discussion about documentation priorities (Classic vs BiDi), this PR contains work that's already complete and ready for review. These pages can be reviewed and merged while that discussion is being finalized.

Note about example code blocks

  • Back, Forward, and Refresh pages: These use Python examples using selenium, to keep them consistent with code blocks on our existing command pages (example).
  • Get Title page: For this, I'm trying out example code using curl and HTTP protocol format (based on a suggestion from @whimboo).

I would appreciate feedback to understand which of these two formats is better for our audience.

Motivation

To improve WebDriver documentation coverage on MDN

Related links

Spec links:

@dipikabh dipikabh requested a review from a team February 12, 2026 18:30
@dipikabh dipikabh requested a review from a team as a code owner February 12, 2026 18:30
@dipikabh dipikabh requested review from pepelsbey and removed request for a team February 12, 2026 18:30
@github-actions github-actions bot added Content:WebDriver WebDriver docs size/m [PR only] 51-500 LoC changed labels Feb 12, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 12, 2026

Copy link
Member

@pepelsbey pepelsbey 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! Thank you 🙂

Just one suggestion!

Copy link
Contributor

@whimboo whimboo left a comment

Choose a reason for hiding this comment

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

Thank you @dipikabh for the first PR for WebDriver classic. I've taken a look and have a couple of comments.

@dipikabh
Copy link
Contributor Author

@pepelsbey @whimboo thank you both for reviewing!

I've addressed the comments and added the New Session command page.
Could you take another look? 🙏

Copy link
Member

@pepelsbey pepelsbey left a comment

Choose a reason for hiding this comment

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

Looks perfect to me!

@dipikabh dipikabh requested a review from whimboo February 26, 2026 03:23
Copy link
Contributor

@whimboo whimboo left a comment

Choose a reason for hiding this comment

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

Looks fine. Maybe you can update the commit message a bit so that it is clear that this PR is not about all navigation commands given that the most important Navigate To command is not included.

Thanks!

@dipikabh dipikabh changed the title [Technical Review] Add pages for navigation commands in WebDriver classic [WebDriver Classic] Add pages for Back, Forward, Refresh, Get Title, and New Session commands Feb 27, 2026
@dipikabh
Copy link
Contributor Author

Maybe you can update the commit message a bit so that it is clear that this PR is not about all navigation commands given that the most important Navigate To command is not included.

Sure, I've updated the title to clarify that this does not cover all navigation commands

@dipikabh
Copy link
Contributor Author

Ran linters locally on the five new files and all pass.
The CI failure is a known workflow bug and is being fixed in #43282.

@dipikabh dipikabh merged commit ffd9f9b into mdn:main Feb 27, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:WebDriver WebDriver docs size/m [PR only] 51-500 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants