Skip to content

[cherry-pick] Fix Agents welcome theme import cleanup#314083

Merged
sandy081 merged 1 commit intorelease/1.119from
cherry-pick/314070
May 4, 2026
Merged

[cherry-pick] Fix Agents welcome theme import cleanup#314083
sandy081 merged 1 commit intorelease/1.119from
cherry-pick/314070

Conversation

@vs-code-engineering
Copy link
Copy Markdown
Contributor

Cherry-pick of #314070 from main.

Fixes #314052

This fixes the Agents welcome theme picker so applying a parent VS Code custom theme is not undone by the preview cleanup when the walkthrough finishes.

Changes

  • Make the theme preview result extend IDisposable and use dispose() for preview cleanup.
  • Mark the preview as applied as soon as apply() is called, so later disposal does not uninstall the imported theme extension.
  • Dispose the preview when switching back to an onboarding theme or tearing down the welcome theme step.

Copilot AI review requested due to automatic review settings May 4, 2026 12:01
@vs-code-engineering vs-code-engineering Bot added the cherry-pick-artifact Auto-generated cherry-pick PR label May 4, 2026
@vs-code-engineering vs-code-engineering Bot review requested due to automatic review settings May 4, 2026 12:01
@sandy081 sandy081 enabled auto-merge (squash) May 4, 2026 13:04
@sandy081 sandy081 merged commit 18cf222 into release/1.119 May 4, 2026
26 checks passed
@sandy081 sandy081 deleted the cherry-pick/314070 branch May 4, 2026 13:05
@vs-code-engineering vs-code-engineering Bot added this to the 1.119.0 milestone May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick-artifact Auto-generated cherry-pick PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants