FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[dotnet] [bidi] Add image size to capture screenshots by nvborisenko · Pull Request #18016 · SeleniumHQ/selenium · GitHub

Repository navigation

[dotnet] [bidi] Add image size to capture screenshots - #18016

Merged
nvborisenko merged 1 commit into
SeleniumHQ:trunkfrom
nvborisenko:bidi-image-size
Sep 10, 2026
Merged

nvborisenko merged 1 commit into
SeleniumHQ:trunkfrom
nvborisenko:bidi-image-size

Conversation

Copy link
Copy Markdown
Member

https://w3c.github.io/webdriver-bidi/#command-browsingContext-captureScreenshot

New imageSize optional parameter.

💥 What does this PR do?

This pull request enhances the screenshot capturing functionality in the BiDi WebDriver by allowing callers to specify a maximum image size for screenshots. The changes introduce a new ImageSize option and update the relevant data structures and method signatures to support this feature.

Screenshot capture enhancements:

  • Added an optional ImageSize property to the CaptureScreenshotOptions record, allowing users to specify MaxWidth and MaxHeight for screenshots. [1] [2]
  • Updated the CaptureScreenshotParameters record and the CaptureScreenshotAsync method to accept and pass through the new ImageSize parameter. [1] [2]

🤖 AI assistance

  • No substantial AI assistance used
  • AI assisted (complete below)
    • Tool(s):
    • What was generated:
    • I reviewed all AI output and can explain the change

💡 Additional Considerations

🔄 Types of changes

  • Cleanup (formatting, renaming)
  • Bug fix (backwards compatible)
  • New feature (non-breaking change which adds functionality and tests!)
  • Breaking change (fix or feature that would cause existing functionality to change)

Copy link
Copy Markdown
Contributor

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

selenium-ci added the C-dotnet .NET Bindings label Sep 10, 2026
nvborisenko merged commit 6724d95 into SeleniumHQ:trunk Sep 10, 2026
25 checks passed
nvborisenko deleted the bidi-image-size branch September 10, 2026 08:34
This was referenced Oct 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-dotnet .NET Bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL