fix(cf): error message does not disappear after update the icon set c…

…onfiguration(#480)
dream-num · Mar 26, 2024 · f17cbf4 · f17cbf4
1 parent d285526
commit f17cbf4
Showing 1 changed file with 15 additions and 15 deletions.
diff --git a/packages/sheets-conditional-format/src/components/panel/rule-edit/iconSet.tsx b/packages/sheets-conditional-format/src/components/panel/rule-edit/iconSet.tsx
@@ -335,21 +335,6 @@ export const IconSet = (props: IStyleEditorProps<unknown, IIconSet>) => {
         );
     }, [configList]);
 
-    const handleClickIconList = (iconType: IIconType) => {
-        currentIconTypeSet(iconType);
-        const list = iconMap[iconType] || [];
-        const config = new Array(list.length).fill('').map((_e, index, list) => createDefaultConfigItem(iconType, index, list));
-        configListSet(config);
-    };
-
-    const handleChange = (keys: string[], v: unknown) => {
-        const oldV = get(configList, keys);
-        if (oldV !== v) {
-            set(configList, keys, v);
-            configListSet([...configList]);
-            errorMapSet(checkResult(configList));
-        }
-    };
     const checkResult = (_configList: typeof configList) => {
         const isTypeSame = _configList.reduce((pre, cur, index) => {
             if ((pre.preType && !pre.result) || _configList.length - 1 === index) {
@@ -391,6 +376,21 @@ export const IconSet = (props: IStyleEditorProps<unknown, IIconSet>) => {
         }
         return {};
     };
+    const handleChange = (keys: string[], v: unknown) => {
+        const oldV = get(configList, keys);
+        if (oldV !== v) {
+            set(configList, keys, v);
+            configListSet([...configList]);
+            errorMapSet(checkResult(configList));
+        }
+    };
+    const handleClickIconList = (iconType: IIconType) => {
+        currentIconTypeSet(iconType);
+        const list = iconMap[iconType] || [];
+        const config = new Array(list.length).fill('').map((_e, index, list) => createDefaultConfigItem(iconType, index, list));
+        configListSet(config);
+        errorMapSet(checkResult(config));
+    };
 
     useEffect(() => {
         const dispose = interceptorManager.intercept(interceptorManager.getInterceptPoints().submit, { handler() {