Fix Diagram Grid Layout inconsistent layout at card removal (#3037)

* fix(DGL): Do not pop because it removes the entry from the original array.
Just check the size without modification of the array.

Signed-off-by: sBouzols <[email protected]>

Author: sBouzols

src/components/diagrams/diagram-grid-layout.tsx

@@ -136,14 +136,7 @@ function DiagramGridLayout({ studyUuid, showInSpreadsheet, visible }: Readonly<D
     };
 
     const removeLayoutItem = (cardUuid: UUID) => {
-        setLayouts((old_layouts) => {
-            const oldLayoutsEntries = Object.entries(old_layouts);
-            if (oldLayoutsEntries.pop()?.[1].length === 2) {
-                return initialLayouts; // Reset to initial layouts if no diagrams left
-            }
-            const newLayoutsEntries = removeInLayoutEntries(oldLayoutsEntries, cardUuid);
-            return Object.fromEntries(newLayoutsEntries);
-        });
+        setLayouts((old_layouts) => Object.fromEntries(removeInLayoutEntries(Object.entries(old_layouts), cardUuid)));
     };
 
     const stopDiagramBlinking = useCallback((diagramUuid: UUID) => {