-
Notifications
You must be signed in to change notification settings - Fork 33
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
fix(menu-list): 修复 左侧菜单全部展开后看不到全部菜单 #89 #99
Conversation
✅ Deploy Preview for vue-hbs-admin ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
.gitignore
Outdated
@@ -6,6 +6,7 @@ yarn-debug.log* | |||
yarn-error.log* | |||
pnpm-debug.log* | |||
lerna-debug.log* | |||
yarn.lock |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
更建议使用 pnpm
|
||
const getMenuClass = computed(() => { | ||
return ['menu-class', 'menu-class--fixed'] | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
没看明白这个 computed 存在的意义
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里是 参照 /src/layouts/default/sidebar/index.vue 的写法。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sidebar/index.vue 是想后面做扩展哈哈~
|
||
const getMenuClass = computed(() => { | ||
return ['menu-class', 'menu-class--fixed'] | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里不需要用computed属性
Description
修复 左侧菜单全部展开后看不到全部菜单
close #89
What is the purpose of this pull request?