Merge pull request #568 from Original-Recipe/feat-topViewVisibleOptimize

fix: Fix the issue of modifying properties after selecting a rectangle

packages/lb-annotation/src/core/toolOperation/pointCloud2DRectOperation.ts

@@ -6,9 +6,9 @@
 
 import { IPointCloud2DRectOperationViewRect } from '@labelbee/lb-utils';
 import EKeyCode from '@/constant/keyCode';
+import AxisUtils from '@/utils/tool/AxisUtils';
 import { RectOperation } from './rectOperation';
 import reCalcRect from './utils/reCalcRect';
-import AxisUtils from '@/utils/tool/AxisUtils';
 
 class PointCloud2DRectOperation extends RectOperation {
   // Whether it is in check mode

packages/lb-components/src/components/pointCloud2DRectOperationView/index.tsx

@@ -83,12 +83,17 @@ const PointCloud2DRectOperationView = (props: IPointCloud2DRectOperationViewProp
   const [loading, setLoading] = useState(true);
 
   const [rightClickRectId, setRightClickRectId] = useState<string>('');
+  // Peugeot used to record the status of boxes that need to be updated
+  const [isMemoryChange, setIsMemoryChange] = useState<boolean>(false);
 
   const rectListInImage = useMemo(
     () => rectList?.filter((item: IPointCloudBoxRect) => item.imageName === mappingData?.path),
     [mappingData?.path, rectList],
   );
 
+  // Save the previous rectListSnImage using useRef
+  const prevRectListRef = useRef(rectListInImage);
+
   const mappingDataPath = useLatest(mappingData?.path);
 
   const recoverSelectedIds = useCallback(
@@ -248,6 +253,15 @@ const PointCloud2DRectOperationView = (props: IPointCloud2DRectOperationViewProp
     if (rightClickRectId) {
       operation.current?.setSelectedRectID(rightClickRectId);
       setRightClickRectId('');
+    } else {
+      /**
+       *  In addition to right-click selection, special scenarios require recording the state of the box before resetting,
+       *  The setResult method will reset the state of all boxes in the current instance, and then use the setSelectedRectID method to reset the state of the previous boxes
+       */
+      if (isMemoryChange && selectedRectID) {
+        operation.current?.setSelectedRectID(selectedRectID);
+        setIsMemoryChange(false);
+      }
     }
   });
 
@@ -357,11 +371,23 @@ const PointCloud2DRectOperationView = (props: IPointCloud2DRectOperationViewProp
         ];
       });
     }
+    // When the attribute changes, it is necessary to record the status of the previous clicks
+    setIsMemoryChange(true);
     updateRectList();
   }, [defaultAttribute]);
 
   useEffect(() => {
-    updateRectList();
+    const prevRectList = prevRectListRef.current;
+    /**
+     *  When there are changes related to box content such as deletion, movement, size change, etc.,
+     *  it is necessary to record the previous state and maintain it
+     */
+    if (!_.isEqual(prevRectList, rectListInImage)) {
+      setIsMemoryChange(true);
+      updateRectList();
+      // Record the latest list of rectangles
+      prevRectListRef.current = rectListInImage;
+    }
   }, [rectListInImage]);
 
   useEffect(() => {