perf(table): remove ExpandIcon

vben
1 parent 81560fd3
Showing 8 changed files with 3 additions and 78 deletions
build/vite/optimizer.ts
build/vite/plugin/hmr.ts
src/components/Table/src/BasicTable.vue
src/components/Table/src/components/ExpandIcon.tsx
src/components/Table/src/hooks/useRowSelection.ts
src/components/Table/src/props.ts
src/components/Tree/src/Tree.vue
src/components/Tree/src/TreeHeader.vue
-// TODO
-import type { GetManualChunk } from 'rollup';
-
-//
-const vendorLibs: { match: string[]; output: string }[] = [
-  // {
-  //   match: ['xlsx'],
-  //   output: 'xlsx',
-  // },
-];
-
-// @ts-ignore
-export const configManualChunk: GetManualChunk = (id: string) => {
-  if (/[\\/]node_modules[\\/]/.test(id)) {
-    const matchItem = vendorLibs.find((item) => {
-      const reg = new RegExp(`[\\/]node_modules[\\/]_?(${item.match.join('|')})(.*)`, 'ig');
-      return reg.test(id);
-    });
-    return matchItem ? matchItem.output : null;
-  }
-};
-import type { Plugin } from 'vite';
-
-/**
- * TODO
- * Temporarily solve the Vite circular dependency problem, and wait for a better solution to fix it later. I don't know what problems this writing will bring.
- * @returns
- */
-
-export function configHmrPlugin(): Plugin {
-  return {
-    name: 'singleHMR',
-    // handleHotUpdate({ modules, file }) {
-    //   if (file.match(/xml$/)) return [];
-
-    //   modules.forEach((m) => {
-    //     if (!m.url.match(/\.(css|less)/)) {
-    //       m.importedModules = new Set();
-    //       m.importers = new Set();
-    //     }
-    //   });
-
-    //   return modules;
-    // },
-  };
-}
@@ -43,7 +43,6 @@
   import { Table } from 'ant-design-vue';
   import { BasicForm, useForm } from '/@/components/Form/index';
   import { PageWrapperFixedHeightKey } from '/@/components/Page';
-  import expandIcon from './components/ExpandIcon';
   import HeaderCell from './components/HeaderCell.vue';
   import { InnerHandlers } from './types/table';
@@ -222,10 +221,8 @@
       const getBindValues = computed(() => {
         const dataSource = unref(getDataSourceRef);
         let propsData: Recordable = {
-          // ...(dataSource.length === 0 ? { getPopupContainer: () => document.body } : {}),
           ...attrs,
           customRow,
-          expandIcon: slots.expandIcon ? null : expandIcon(),
           ...unref(getProps),
           ...unref(getHeaderProps),
           scroll: unref(getScrollRef),
-import { BasicArrow } from '/@/components/Basic';
-
-export default () => {
-  return (props: Recordable) => {
-    if (!props.expandable) {
-      if (props.needIndentSpaced) {
-        return <span class="ant-table-row-expand-icon ant-table-row-spaced" />;
-      } else {
-        return <span />;
-      }
-    }
-    return (
-      <BasicArrow
-        style="margin-right: 8px"
-        iconStyle="margin-top: -2px;"
-        onClick={(e: Event) => {
-          props.onExpand(props.record, e);
-        }}
-        expand={props.expanded}
-      />
-    );
-  };
-};
@@ -21,11 +21,8 @@ export function useRowSelection(
     return {
       selectedRowKeys: unref(selectedRowKeysRef),
-      hideDefaultSelections: false,
       onChange: (selectedRowKeys: string[]) => {
         setSelectedRowKeys(selectedRowKeys);
-        // selectedRowKeysRef.value = selectedRowKeys;
-        // selectedRowRef.value = selectedRows;
       },
       ...omit(rowSelection, ['onChange']),
     };
@@ -10,6 +10,7 @@ import type {
   SizeType,
 } from './types/table';
 import type { FormProps } from '/@/components/Form';
+
 import { DEFAULT_FILTER_FN, DEFAULT_SORT_FN, FETCH_SETTING, DEFAULT_SIZE } from './const';
 import { propTypes } from '/@/utils/propTypes';
@@ -13,6 +13,7 @@
     watch,
     onMounted,
   } from 'vue';
+  import TreeHeader from './TreeHeader.vue';
   import { Tree, Empty } from 'ant-design-vue';
   import { TreeIcon } from './TreeIcon';
   import { ScrollContainer } from '/@/components/Container';
@@ -23,7 +24,6 @@
   import { useTree } from './useTree';
   import { useContextMenu } from '/@/hooks/web/useContextMenu';
   import { CreateContextOptions } from '/@/components/ContextMenu';
-  import TreeHeader from './TreeHeader.vue';
   import { treeEmits, treeProps } from './tree';
   import { createBEM } from '/@/utils/bem';
@@ -5,7 +5,7 @@
       {{ title }}
     </BasicTitle>
     <div
-      class="flex flex-1 justify-self-stretch items-center cursor-pointer"
+      class="flex items-center flex-1 cursor-pointer justify-self-stretch"
       v-if="search || toolbar"
     >
       <div :class="getInputSearchCls" v-if="search">
@@ -46,7 +46,6 @@
   const [bem] = createBEM('tree-header');
-  // eslint-disable vue/valid-define-emits
   const props = defineProps({
     helpMessage: {
       type: [String, Array] as PropType<string | string[]>,