[3645] Propagate a pseudo selection change when re-selecting a repres…

…entation Bug: #3645 Signed-off-by: Pierre-Charles David <[email protected]>
eclipse-sirius · Jun 19, 2024 · fbad3ee · fbad3ee
1 parent 9ebb240
commit fbad3ee
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.adoc b/CHANGELOG.adoc
@@ -58,6 +58,7 @@ More existing APIs will be migrated to this new common pattern.
 - https://github.com/eclipse-sirius/sirius-web/issues/3628[#3628] [sirius-web] Restore support for expand all and reveal in the explorer
 - https://github.com/eclipse-sirius/sirius-web/issues/3616[#3616] [diagram] Fix potential exceptions due to duplicate keys in diagram event processing
 - https://github.com/eclipse-sirius/sirius-web/issues/3624[#3624] [diagram] Fix an issue where the header separator does not fill the entire width of the node
+- https://github.com/eclipse-sirius/sirius-web/issues/3645[#3645] [core] Fix a regression where clicking on an already selected representation did not re-open it
 
 === New Features
 

diff --git a/packages/core/frontend/sirius-components-core/src/selection/SelectionContext.tsx b/packages/core/frontend/sirius-components-core/src/selection/SelectionContext.tsx
@@ -41,7 +41,10 @@ export const SelectionContextProvider = ({ initialSelection, children }: Selecti
         .map((entry) => entry.id)
         .sort()
         .join(':');
-      if (prevSelectionKey !== newSelectionKey) {
+      if (
+        prevSelectionKey !== newSelectionKey ||
+        selection.entries.find((entry) => entry.kind.startsWith('siriusComponents://representation?type='))
+      ) {
         return { ...prevState, selection };
       } else {
         return prevState;