-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
when using Multiple Sidebars
the sidebar not being updated
#809
Comments
brc-dd
added
bug
Something isn't working
theme
Related to the theme
and removed
bug: pending triage
Maybe a bug, waiting for confirmation
labels
Jun 18, 2022
Fix: vitepress/src/client/theme-default/components/VPSidebarGroup.vue Lines 15 to 21 in 107724a
But what will be the |
add a watcher or just add a setter to the computed: https://vuejs.org/guide/essentials/computed.html#writable-computed |
@brc-dd try adding this: watchEffect(() => {
collapsed.value = props.collapsible && props.collapsed
}) |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Describe the bug
just read the title
Reproduction
https://github.com/antfu/vite-plugin-pwa/tree/userquin/feat-new-theme
you can also disable pwa, just add
disable: true
option onVitePWA
plugin here: https://github.com/antfu/vite-plugin-pwa/blob/userquin/feat-new-theme/docs/vite.config.ts#L53from root folder run
pnpm install && pnpm run build
thenpnpm run docs
to start dev orpnpm run docs:preview
to start previewExpected behavior
The sidebar should update
System Info
Additional context
No response
Validations
The text was updated successfully, but these errors were encountered: