fix: 🐛 tag/size 搜索条件相关问题

meetqy · Mar 6, 2023 · 2e601e5 · 2e601e5
1 parent 7de46bf
commit 2e601e5
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 27 deletions.
diff --git a/components/JustifyLayout/SearchModule/Size.tsx b/components/JustifyLayout/SearchModule/Size.tsx
@@ -1,31 +1,25 @@
 import { CaretDownOutlined } from "@ant-design/icons";
 import { Col, InputNumber, Popover, Row, Typography } from "antd";
-import _ from "lodash";
-import { useEffect, useMemo, useState } from "react";
+import { useMemo, useState } from "react";
 
 interface Props {
   value: EagleUse.SearchParams["size"];
   onChange: (value: EagleUse.SearchParams["size"]) => void;
 }
 
 const Size = (props: Props) => {
-  const [value, setValue] = useState<EagleUse.SearchParams["size"]>({
-    width: {
-      min: 0,
-      max: 0,
-    },
-    height: {
-      min: 0,
-      max: 0,
-    },
-  });
-
-  useEffect(() => {
-    if (_.isEqual(props.value, value)) return;
-    if (!props.value) return;
-
-    setValue(props.value);
-  }, [props.value, value]);
+  const [value, setValue] = useState<EagleUse.SearchParams["size"]>(
+    props.value || {
+      width: {
+        min: 0,
+        max: 0,
+      },
+      height: {
+        min: 0,
+        max: 0,
+      },
+    }
+  );
 
   const changeValue = (
     e: number,

diff --git a/components/JustifyLayout/SearchModule/Tag.tsx b/components/JustifyLayout/SearchModule/Tag.tsx
@@ -1,7 +1,6 @@
 import { tagsState } from "@/store";
 import { Select } from "antd";
-import _ from "lodash";
-import { useEffect, useState } from "react";
+import { useState } from "react";
 import { useRecoilValue } from "recoil";
 interface Props {
   value: string[];
@@ -10,12 +9,7 @@ interface Props {
 
 const Tag = (props: Props) => {
   const tags = useRecoilValue(tagsState);
-  const [value, setValue] = useState<string[]>([]);
-
-  useEffect(() => {
-    if (_.isEqual(props.value, value)) return;
-    setValue(props.value);
-  }, [props.value, value]);
+  const [value, setValue] = useState<string[]>(props.value || []);
 
   const onChange = (value: string[]) => {
     setValue(value);