Skip to content

fix(core): preserve object-form dock icons in command projection#378

Merged
antfu merged 1 commit into
vitejs:mainfrom
SaKaNa-Y:fix/dock-command-icon-object-form
Jun 24, 2026
Merged

fix(core): preserve object-form dock icons in command projection#378
antfu merged 1 commit into
vitejs:mainfrom
SaKaNa-Y:fix/dock-command-icon-object-form

Conversation

@SaKaNa-Y

Copy link
Copy Markdown
Contributor

Description

Vite+ icon not rendering correctly, Because the Vite+ dock icon is passed as an object. Since I’m not sure whether it was intended for future theme support, I avoided changing the registration and instead fixed toCommand.

Linked Issues

Additional context

@pkg-pr-new

pkg-pr-new Bot commented Jun 12, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@vitejs/devtools@378
npm i https://pkg.pr.new/@vitejs/devtools-kit@378
npm i https://pkg.pr.new/@vitejs/devtools-rolldown@378
npm i https://pkg.pr.new/@vitejs/devtools-self-inspect@378

commit: 9bbd0a3

@antfu

antfu commented Jun 12, 2026

Copy link
Copy Markdown
Member

I think instead we should make the command to accept object shape icon

@SaKaNa-Y

Copy link
Copy Markdown
Contributor Author

I know, so we should first modify the type files in the devframe repository. I tried modifying it.devframes/devframe#32

@antfu antfu merged commit db10e17 into vitejs:main Jun 24, 2026
7 checks passed
@SaKaNa-Y SaKaNa-Y deleted the fix/dock-command-icon-object-form branch June 24, 2026 02:55
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