fix(menu): fix hideMenu not working close #338

vbenjs · Mar 9, 2021 · 5b2fbfb · 5b2fbfb
1 parent 53867a8
commit 5b2fbfb
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 9 deletions.
diff --git a/src/components/Menu/src/components/BasicSubMenuItem.vue b/src/components/Menu/src/components/BasicSubMenuItem.vue
@@ -36,9 +36,7 @@
     setup(props) {
       const { prefixCls } = useDesign('basic-menu-item');
 
-      const getShowMenu = computed(() => {
-        return !props.item.meta?.hideMenu;
-      });
+      const getShowMenu = computed(() => !props.item.meta?.hideMenu);
       function menuHasChildren(menuTreeItem: MenuType): boolean {
         return (
           Reflect.has(menuTreeItem, 'children') &&

diff --git a/src/components/SimpleMenu/src/SimpleMenu.vue b/src/components/SimpleMenu/src/SimpleMenu.vue
@@ -94,7 +94,7 @@
       listenerLastChangeTab((route) => {
         if (route.name === REDIRECT_NAME) return;
 
-        currentActiveMenu.value = route.meta?.currentActiveMenu;
+        currentActiveMenu.value = route.meta?.currentActiveMenu as string;
         handleMenuChange(route);
 
         if (unref(currentActiveMenu)) {

diff --git a/src/components/SimpleMenu/src/SimpleSubMenu.vue b/src/components/SimpleMenu/src/SimpleSubMenu.vue
@@ -75,10 +75,7 @@
       const { t } = useI18n();
       const { prefixCls } = useDesign('simple-menu');
 
-      const getShowMenu = computed(() => {
-        return !props.item?.hideMenu;
-      });
-
+      const getShowMenu = computed(() => !props.item?.meta?.hideMenu);
       const getIcon = computed(() => props.item?.icon);
       const getI18nName = computed(() => t(props.item?.name));
       const getShowSubTitle = computed(() => !props.collapse || !props.parent);

diff --git a/src/router/routes/modules/demo/feat.ts b/src/router/routes/modules/demo/feat.ts
@@ -92,7 +92,7 @@ const feat: AppRouteModule = {
                 currentActiveMenu: '/feat/breadcrumb/children',
                 title: t('routes.demo.feat.breadcrumbChildrenDetail'),
                 hideTab: true,
-                hideMenu: true,
+                // hideMenu: true,
               },
             },
           ],