[3595] Only update the selection when it really changes

Bug: #3595
Signed-off-by: Pierre-Charles David <[email protected]>

CHANGELOG.adoc

@@ -66,6 +66,7 @@
 - https://github.com/eclipse-sirius/sirius-web/issues/3552[#3552] [core] Rewrite EObjectIdManager using lower-level code for performance
 - https://github.com/eclipse-sirius/sirius-web/issues/3555[#3555] [core] Avoid costly recomputations in RepresentationMigrationService
 - https://github.com/eclipse-sirius/sirius-web/issues/3558[#3558] [core] Avoid all representation migration overhead when not needed
+- https://github.com/eclipse-sirius/sirius-web/issues/3595[#3595] [core] Avoid re-rendering the workbench when the selection does not really change
 
 == v2024.5.0
 

packages/core/frontend/sirius-components-core/src/selection/SelectionContext.tsx

@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2023 Obeo.
+ * Copyright (c) 2023, 2024 Obeo.
  * This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v2.0
  * which accompanies this distribution, and is available at
@@ -32,7 +32,21 @@ export const SelectionContextProvider = ({ initialSelection, children }: Selecti
   });
 
   const setSelection = useCallback((selection: Selection) => {
-    setState((prevState) => ({ ...prevState, selection }));
+    setState((prevState) => {
+      const prevSelectionKey = prevState.selection.entries
+        .map((entry) => entry.id)
+        .sort()
+        .join(':');
+      const newSelectionKey = selection.entries
+        .map((entry) => entry.id)
+        .sort()
+        .join(':');
+      if (prevSelectionKey !== newSelectionKey) {
+        return { ...prevState, selection };
+      } else {
+        return prevState;
+      }
+    });
   }, []);
 
   return (