From ee1c3498587951a6a4cc0b49edb9dacf3f2af5c3 Mon Sep 17 00:00:00 2001
From: Vben <anncwb@126.com>
Date: Thu, 8 Apr 2021 23:04:16 +0800
Subject: [PATCH] fix(menu): improve menu logic, fix #461

---
 CHANGELOG.zh_CN.md                            |  1 +
 .../Application/src/AppDarkModeToggle.vue     | 33 +++++++++----------
 src/components/Icon/src/index.vue             |  6 +---
 src/components/SimpleMenu/src/SimpleMenu.vue  | 13 +++++++-
 .../SimpleMenu/src/components/Menu.vue        |  9 +++++
 .../SimpleMenu/src/components/MenuItem.vue    |  9 +++--
 .../SimpleMenu/src/components/SubMenuItem.vue | 13 ++++++--
 src/components/SimpleMenu/src/useOpenKeys.ts  |  2 +-
 src/layouts/default/menu/index.vue            |  3 +-
 src/layouts/default/setting/SettingDrawer.tsx |  2 +-
 src/layouts/default/tabs/index.less           |  2 +-
 src/router/routes/modules/demo/charts.ts      |  2 +-
 src/views/sys/login/Login.vue                 |  4 +++
 13 files changed, 66 insertions(+), 33 deletions(-)

diff --git a/CHANGELOG.zh_CN.md b/CHANGELOG.zh_CN.md
index 919c1bf3..e7c15efe 100644
--- a/CHANGELOG.zh_CN.md
+++ b/CHANGELOG.zh_CN.md
@@ -3,6 +3,7 @@
 ### 🐛 Bug Fixes
 
 - 登录页样式修复
+- 修复菜单已知问题
 
 ## 2.2.0 (2021-04-06)
 
diff --git a/src/components/Application/src/AppDarkModeToggle.vue b/src/components/Application/src/AppDarkModeToggle.vue
index fa7e892a..20c7b4ad 100644
--- a/src/components/Application/src/AppDarkModeToggle.vue
+++ b/src/components/Application/src/AppDarkModeToggle.vue
@@ -3,7 +3,6 @@
     v-if="getShowDarkModeToggle"
     :class="[
       prefixCls,
-      `${prefixCls}--${size}`,
       {
         [`${prefixCls}--dark`]: isDark,
       },
@@ -30,13 +29,13 @@
   export default defineComponent({
     name: 'DarkModeToggle',
     components: { SvgIcon },
-    props: {
-      size: {
-        type: String,
-        default: 'default',
-        validate: (val) => ['default', 'large'].includes(val),
-      },
-    },
+    // props: {
+    //   size: {
+    //     type: String,
+    //     default: 'default',
+    //     validate: (val) => ['default', 'large'].includes(val),
+    //   },
+    // },
     setup() {
       const { prefixCls } = useDesign('dark-mode-toggle');
       const { getDarkMode, setDarkMode, getShowDarkModeToggle } = useRootSetting();
@@ -97,15 +96,15 @@
       }
     }
 
-    &--large {
-      width: 72px;
-      height: 34px;
-      padding: 0 10px;
+    // &--large {
+    //   width: 70px;
+    //   height: 34px;
+    //   padding: 0 10px;
 
-      .@{prefix-cls}-inner {
-        width: 26px;
-        height: 26px;
-      }
-    }
+    //   .@{prefix-cls}-inner {
+    //     width: 26px;
+    //     height: 26px;
+    //   }
+    // }
   }
 </style>
diff --git a/src/components/Icon/src/index.vue b/src/components/Icon/src/index.vue
index ae36707d..c76a94f9 100644
--- a/src/components/Icon/src/index.vue
+++ b/src/components/Icon/src/index.vue
@@ -24,8 +24,6 @@
   import Iconify from '@purge-icons/generated';
   import { isString } from '/@/utils/is';
   import { propTypes } from '/@/utils/propTypes';
-  import { useRootSetting } from '/@/hooks/setting/useRootSetting';
-  import { ThemeEnum } from '/@/enums/appEnum';
 
   const SVG_END_WITH_FLAG = '|svg';
   export default defineComponent({
@@ -46,8 +44,6 @@
     setup(props) {
       const elRef = ref<ElRef>(null);
 
-      const { getDarkMode } = useRootSetting();
-
       const isSvgIcon = computed(() => props.icon?.endsWith(SVG_END_WITH_FLAG));
       const getSvgIcon = computed(() => props.icon.replace(SVG_END_WITH_FLAG, ''));
       const getIconRef = computed(() => `${props.prefix ? props.prefix + ':' : ''}${props.icon}`);
@@ -85,7 +81,7 @@
 
           return {
             fontSize: `${fs}px`,
-            color: color || (unref(getDarkMode) === ThemeEnum.DARK ? '#fff' : '#303133'),
+            color: color,
             display: 'inline-flex',
           };
         }
diff --git a/src/components/SimpleMenu/src/SimpleMenu.vue b/src/components/SimpleMenu/src/SimpleMenu.vue
index 1d3c93bf..77d67d6e 100644
--- a/src/components/SimpleMenu/src/SimpleMenu.vue
+++ b/src/components/SimpleMenu/src/SimpleMenu.vue
@@ -1,11 +1,12 @@
 <template>
   <Menu
     v-bind="getBindValues"
-    @select="handleSelect"
     :activeName="activeName"
     :openNames="getOpenKeys"
     :class="prefixCls"
     :activeSubMenuNames="activeSubMenuNames"
+    @select="handleSelect"
+    @open-change="handleOpenChange"
   >
     <template v-for="item in items" :key="item.path">
       <SimpleSubMenu
@@ -53,6 +54,7 @@
       beforeClickFn: {
         type: Function as PropType<(key: string) => Promise<boolean>>,
       },
+      isSplitMenu: propTypes.bool,
     },
     emits: ['menuClick'],
     setup(props, { attrs, emit }) {
@@ -94,6 +96,9 @@
       watch(
         () => props.items,
         () => {
+          if (!props.isSplitMenu) {
+            return;
+          }
           setOpenKeys(currentRoute.value.path);
         },
         { flush: 'post' }
@@ -135,11 +140,17 @@
         menuState.activeName = key;
       }
 
+      function handleOpenChange(v) {
+        console.log('======================');
+        console.log(v);
+        console.log('======================');
+      }
       return {
         prefixCls,
         getBindValues,
         handleSelect,
         getOpenKeys,
+        handleOpenChange,
         ...toRefs(menuState),
       };
     },
diff --git a/src/components/SimpleMenu/src/components/Menu.vue b/src/components/SimpleMenu/src/components/Menu.vue
index 8e352f05..3468dc20 100644
--- a/src/components/SimpleMenu/src/components/Menu.vue
+++ b/src/components/SimpleMenu/src/components/Menu.vue
@@ -138,6 +138,15 @@
           });
           emit('select', name);
         });
+
+        rootMenuEmitter.on('open-name-change', ({ name, opened }) => {
+          if (opened && !openedNames.value.includes(name)) {
+            openedNames.value.push(name);
+          } else if (!opened) {
+            const index = openedNames.value.findIndex((item) => item === name);
+            index !== -1 && openedNames.value.splice(index, 1);
+          }
+        });
       });
 
       return { getClass, openedNames };
diff --git a/src/components/SimpleMenu/src/components/MenuItem.vue b/src/components/SimpleMenu/src/components/MenuItem.vue
index 55d8e6fa..15d5534e 100644
--- a/src/components/SimpleMenu/src/components/MenuItem.vue
+++ b/src/components/SimpleMenu/src/components/MenuItem.vue
@@ -66,11 +66,16 @@
 
       function handleClickItem() {
         const { disabled } = props;
-        if (disabled) return;
+        if (disabled) {
+          return;
+        }
 
         rootMenuEmitter.emit('on-menu-item-select', props.name);
-        if (unref(getCollapse)) return;
+        if (unref(getCollapse)) {
+          return;
+        }
         const { uidList } = getParentList();
+
         rootMenuEmitter.emit('on-update-opened', {
           opend: false,
           parent: instance?.parent,
diff --git a/src/components/SimpleMenu/src/components/SubMenuItem.vue b/src/components/SimpleMenu/src/components/SubMenuItem.vue
index d3f1853d..0d24cbd7 100644
--- a/src/components/SimpleMenu/src/components/SubMenuItem.vue
+++ b/src/components/SimpleMenu/src/components/SubMenuItem.vue
@@ -43,8 +43,9 @@
           :class="`${prefixCls}-submenu-title-icon`"
         />
       </div>
-      <template #content>
-        <div v-bind="getEvents(true)" v-show="opened">
+      <!-- eslint-disable-next-line -->
+      <template #content v-show="opened">
+        <div v-bind="getEvents(true)">
           <ul :class="[prefixCls, `${prefixCls}-${getTheme}`, `${prefixCls}-popup`]">
             <slot></slot>
           </ul>
@@ -78,7 +79,7 @@
   import { isBoolean, isObject } from '/@/utils/is';
   import Mitt from '/@/utils/mitt';
 
-  const DELAY = 250;
+  const DELAY = 200;
   export default defineComponent({
     name: 'SubMenu',
     components: {
@@ -189,6 +190,7 @@
         const { disabled } = props;
         if (disabled || unref(getCollapse)) return;
         const opened = state.opened;
+
         if (unref(getAccordion)) {
           const { uidList } = getParentList();
           rootMenuEmitter.emit('on-update-opened', {
@@ -196,6 +198,11 @@
             parent: instance?.parent,
             uidList: uidList,
           });
+        } else {
+          rootMenuEmitter.emit('open-name-change', {
+            name: props.name,
+            opened: !opened,
+          });
         }
         state.opened = !opened;
       }
diff --git a/src/components/SimpleMenu/src/useOpenKeys.ts b/src/components/SimpleMenu/src/useOpenKeys.ts
index 565da6af..b4a9a510 100644
--- a/src/components/SimpleMenu/src/useOpenKeys.ts
+++ b/src/components/SimpleMenu/src/useOpenKeys.ts
@@ -8,7 +8,7 @@ import { uniq } from 'lodash-es';
 import { getAllParentPath } from '/@/router/helper/menuHelper';
 
 import { useTimeoutFn } from '/@/hooks/core/useTimeout';
-import { useDebounce } from '../../../hooks/core/useDebounce';
+import { useDebounce } from '/@/hooks/core/useDebounce';
 
 export function useOpenKeys(
   menuState: MenuState,
diff --git a/src/layouts/default/menu/index.vue b/src/layouts/default/menu/index.vue
index 2a1f6c19..9e340906 100644
--- a/src/layouts/default/menu/index.vue
+++ b/src/layouts/default/menu/index.vue
@@ -49,6 +49,7 @@
         getAccordion,
         getIsHorizontal,
         getIsSidebarType,
+        getSplit,
       } = useMenuSetting();
       const { getShowLogo } = useRootSetting();
 
@@ -144,7 +145,7 @@
         // console.log(menus);
         if (!menus || !menus.length) return null;
         return !props.isHorizontal ? (
-          <SimpleMenu {...menuProps} items={menus} />
+          <SimpleMenu {...menuProps} isSplitMenu={unref(getSplit)} items={menus} />
         ) : (
           <BasicMenu
             {...menuProps}
diff --git a/src/layouts/default/setting/SettingDrawer.tsx b/src/layouts/default/setting/SettingDrawer.tsx
index b68b8c7e..beba91c9 100644
--- a/src/layouts/default/setting/SettingDrawer.tsx
+++ b/src/layouts/default/setting/SettingDrawer.tsx
@@ -408,7 +408,7 @@ export default defineComponent({
         wrapClassName="setting-drawer"
       >
         {unref(getShowDarkModeToggle) && <Divider>{() => t('layout.setting.darkMode')}</Divider>}
-        {unref(getShowDarkModeToggle) && <AppDarkModeToggle class="mx-auto" size="large" />}
+        {unref(getShowDarkModeToggle) && <AppDarkModeToggle class="mx-auto" />}
         <Divider>{() => t('layout.setting.navMode')}</Divider>
         {renderSidebar()}
         <Divider>{() => t('layout.setting.sysTheme')}</Divider>
diff --git a/src/layouts/default/tabs/index.less b/src/layouts/default/tabs/index.less
index f22c53a9..5ea76a5f 100644
--- a/src/layouts/default/tabs/index.less
+++ b/src/layouts/default/tabs/index.less
@@ -82,7 +82,7 @@ html[data-theme='dark'] {
       .ant-tabs-tab-active {
         position: relative;
         padding-left: 18px;
-        color: @white;
+        color: @white !important;
         background: @primary-color;
         border: 0;
         transition: none;
diff --git a/src/router/routes/modules/demo/charts.ts b/src/router/routes/modules/demo/charts.ts
index f20c9c8b..ac0a58a4 100644
--- a/src/router/routes/modules/demo/charts.ts
+++ b/src/router/routes/modules/demo/charts.ts
@@ -7,7 +7,7 @@ const charts: AppRouteModule = {
   path: '/charts',
   name: 'Charts',
   component: LAYOUT,
-  redirect: '/charts/apexChart',
+  redirect: '/charts/echarts/map',
   meta: {
     icon: 'ion:bar-chart-outline',
     title: t('routes.demo.charts.charts'),
diff --git a/src/views/sys/login/Login.vue b/src/views/sys/login/Login.vue
index 6d8b81f4..225b379c 100644
--- a/src/views/sys/login/Login.vue
+++ b/src/views/sys/login/Login.vue
@@ -112,6 +112,10 @@
       &-form {
         background: transparent !important;
       }
+
+      .app-iconify {
+        color: #fff;
+      }
     }
   }