Skip to content

feat: browser list UI#682

Merged
Skn0tt merged 2 commits intomicrosoft:mainfrom
Skn0tt:browserlist-ui
Sep 1, 2025
Merged

feat: browser list UI#682
Skn0tt merged 2 commits intomicrosoft:mainfrom
Skn0tt:browserlist-ui

Conversation

@Skn0tt
Copy link
Member

@Skn0tt Skn0tt commented Aug 29, 2025

Adds UI for showing a list of browsers, each with a "Pick Locator" and "Close" button. This uses the data from microsoft/playwright#37027. On old versions, we try to infer the browser information from the currently selected project.

Screen.Recording.2025-08-14.at.15.51.47.mov

@Skn0tt Skn0tt requested a review from dgozman August 29, 2025 14:32
@Skn0tt Skn0tt self-assigned this Aug 29, 2025
@Skn0tt Skn0tt mentioned this pull request Aug 29, 2025
@Skn0tt Skn0tt changed the title chore: browser list UI feat: browser list UI Aug 29, 2025
@Skn0tt
Copy link
Member Author

Skn0tt commented Aug 29, 2025

Here's how the SVGs look:

Screenshot 2025-08-29 at 16 57 49

if (browserId)
await this._reusedBrowser.closeBrowser(browserId, 'User requested close from VS Code Extension');
else
this._reusedBrowser.closeAllBrowsers();
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing await?

Copy link
Member Author

Choose a reason for hiding this comment

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

no, this just closes the underlying process and has also been synchronous before.

@Skn0tt Skn0tt merged commit 6663081 into microsoft:main Sep 1, 2025
7 checks passed
Skn0tt added a commit to Skn0tt/playwright-vscode that referenced this pull request Oct 1, 2025
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