Skip to content

Use PNG for Add Element to Chat screenshots#316807

Draft
jruales wants to merge 1 commit into
mainfrom
jruales/2026.05-element-to-chat-png
Draft

Use PNG for Add Element to Chat screenshots#316807
jruales wants to merge 1 commit into
mainfrom
jruales/2026.05-element-to-chat-png

Conversation

@jruales
Copy link
Copy Markdown
Contributor

@jruales jruales commented May 16, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 16, 2026 22:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates Integrated Browser “Add Element to Chat” screenshots to use PNG output, and extends the BrowserView screenshot capture API to support selecting the output image format.

Changes:

  • Add format?: 'jpeg' | 'png' to IBrowserViewCaptureScreenshotOptions (with updated docs for quality behavior).
  • Update Electron main-process screenshot capture to encode as PNG when requested.
  • Switch “Add Element to Chat” element screenshots to request PNG output.
Show a summary per file
File Description
src/vs/workbench/contrib/browserView/electron-browser/features/browserEditorChatFeatures.ts Requests PNG screenshots when attaching selected elements to chat.
src/vs/platform/browserView/electron-main/browserView.ts Encodes captured screenshots as PNG when options.format === 'png', otherwise JPEG.
src/vs/platform/browserView/common/browserView.ts Extends screenshot capture options with format and clarifies quality applicability/default.

Copilot's findings

  • Files reviewed: 3/3 changed files
  • Comments generated: 0

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants