Skip to content

refactor(client): remove code related to view mode#1074

Merged
webfansplz merged 1 commit intovuejs:mainfrom
skirtles-code:remove-view-mode
Apr 21, 2026
Merged

refactor(client): remove code related to view mode#1074
webfansplz merged 1 commit intovuejs:mainfrom
skirtles-code:remove-view-mode

Conversation

@skirtles-code
Copy link
Copy Markdown
Contributor

The file view-mode.ts is empty. I've removed it. It used to contain code for useToggleViewMode.

The file ViewModeSwitch.vue is unused. It contains code that attempts to call useToggleViewMode, even though it doesn't exist. This causes type checking to fail as it can't find the function.

settings.vue also contains a reference to useToggleViewMode. That code was already commented out but I've removed it.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 25, 2026

Deploy Preview for vue-devtools-docs canceled.

Name Link
🔨 Latest commit 0bb8302
🔍 Latest deploy log https://app.netlify.com/projects/vue-devtools-docs/deploys/69c3c1567fcc7700080e30fe

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Mar 25, 2026

Open in StackBlitz

@vue/devtools-applet

npm i https://pkg.pr.new/@vue/devtools-applet@1074

@vue/devtools-core

npm i https://pkg.pr.new/@vue/devtools-core@1074

@vue/devtools

npm i https://pkg.pr.new/@vue/devtools@1074

@vue/devtools-api

npm i https://pkg.pr.new/@vue/devtools-api@1074

@vue/devtools-kit

npm i https://pkg.pr.new/@vue/devtools-kit@1074

@vue/devtools-electron

npm i https://pkg.pr.new/@vue/devtools-electron@1074

@vue/devtools-shared

npm i https://pkg.pr.new/@vue/devtools-shared@1074

@vue/devtools-ui

npm i https://pkg.pr.new/@vue/devtools-ui@1074

vite-plugin-vue-devtools

npm i https://pkg.pr.new/vite-plugin-vue-devtools@1074

commit: 0bb8302

@webfansplz webfansplz merged commit c13452a into vuejs:main Apr 21, 2026
8 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.

2 participants