[4549] Filter edge tools from palette tool sections

Bug: #4549 Signed-off-by: Michaël Charfadi <[email protected]>
eclipse-sirius · Feb 14, 2025 · b6f653d · b6f653d
1 parent a0dd9bd
commit b6f653d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/CHANGELOG.adoc b/CHANGELOG.adoc
@@ -98,6 +98,7 @@ Now they are executed for some data if the data was coming from a studio too.
 - https://github.com/eclipse-sirius/sirius-web/issues/4547[#4547] [vs-code] Adapt the VSCode extension to the use of SemanticData#id as editingContextId
 - https://github.com/eclipse-sirius/sirius-web/issues/4561[#4561] [sirius-web] Display boolean values properly in the query view
 - https://github.com/eclipse-sirius/sirius-web/issues/4522[#4522] [diagram] Fix an issue where custom tools were no longer contributed to the diagram palette
+- https://github.com/eclipse-sirius/sirius-web/issues/4549[#4549] [diagram] Fix an issue where edge tools could be shown in a palette tool section
 
 
 === New Features

diff --git a/...tend/sirius-components-diagrams/src/renderer/palette/tool-list/PaletteToolSectionList.tsx b/...tend/sirius-components-diagrams/src/renderer/palette/tool-list/PaletteToolSectionList.tsx
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2024 Obeo.
+ * Copyright (c) 2024, 2025 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
@@ -16,6 +16,7 @@ import ListItemButton from '@mui/material/ListItemButton';
 import ListItemText from '@mui/material/ListItemText';
 import Tooltip from '@mui/material/Tooltip';
 import { makeStyles } from 'tss-react/mui';
+import { isSingleClickOnDiagramElementTool } from '../Palette';
 import { ToolListItem } from '../tool-list-item/ToolListItem';
 import { PaletteToolSectionListProps } from './PaletteToolSectionList.types';
 
@@ -68,7 +69,7 @@ export const PaletteToolSectionList = ({ toolSection, onToolClick, onBackToMainL
           <ListItemText className={classes.sectionTitleListItemText} primary={toolSection.label} />
         </ListItemButton>
       </Tooltip>
-      {toolSection.tools.map((tool) => (
+      {toolSection.tools.filter(isSingleClickOnDiagramElementTool).map((tool) => (
         <ToolListItem onToolClick={onToolClick} tool={tool} key={tool.id} />
       ))}
     </List>