Skip to content

browser: prevent new tab from flashing in quick pick#304297

Merged
jruales merged 2 commits intomicrosoft:mainfrom
jonathanrao99:fix/304217-browser-tab-flash
Mar 24, 2026
Merged

browser: prevent new tab from flashing in quick pick#304297
jruales merged 2 commits intomicrosoft:mainfrom
jonathanrao99:fix/304217-browser-tab-flash

Conversation

@jonathanrao99
Copy link
Copy Markdown
Contributor

Fixes #304217

Summary

When selecting "New Tab" from the browser tab quick pick, the newly created editor could briefly appear in the quick pick before it closed.

This change hides the quick pick immediately after logging telemetry and before opening the new editor, preventing listeners from rebuilding the items list while the quick pick is still visible.

Testing

  • Manually verified that selecting "New Tab" no longer causes the new tab to briefly appear in the quick pick

@vs-code-engineering vs-code-engineering bot added this to the 1.114.0 milestone Mar 23, 2026
@vs-code-engineering
Copy link
Copy Markdown
Contributor

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@kycutler

Matched files:

  • src/vs/workbench/contrib/browserView/electron-browser/features/browserTabManagementFeatures.ts

@jruales

Matched files:

  • src/vs/workbench/contrib/browserView/electron-browser/features/browserTabManagementFeatures.ts

@jruales jruales enabled auto-merge (squash) March 24, 2026 16:50
@kycutler
Copy link
Copy Markdown
Contributor

Looks great @jonathanrao99, thanks so much!

@jruales jruales merged commit 81d9831 into microsoft:main Mar 24, 2026
18 checks passed
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.

Flashing element when selecting New Tab from Quick Open Browser Tab

3 participants