refactor(core): use removeSelectedNodes in click handler

Signed-off-by: braks <78412429+bcakmakoglu@users.noreply.github.com>
This commit is contained in:
braks
2023-01-18 23:03:17 +01:00
committed by Braks
parent c25059444a
commit f0d4763a99
2 changed files with 5 additions and 5 deletions

View File

@@ -26,7 +26,7 @@ const {
multiSelectionActive,
emits,
findNode,
removeSelectedElements,
removeSelectedNodes,
addSelectedNodes,
updateNodeDimensions,
onUpdateNodeInternals,
@@ -203,7 +203,7 @@ function onDoubleClick(event: MouseEvent) {
function onSelectNode(event: MouseEvent) {
if (selectable && (!selectNodesOnDrag || !draggable)) {
handleNodeClick(node, multiSelectionActive, addSelectedNodes, removeSelectedElements, $$(nodesSelectionActive))
handleNodeClick(node, multiSelectionActive, addSelectedNodes, removeSelectedNodes, $$(nodesSelectionActive))
}
emit.click({ event, node, connectedEdges })
@@ -219,7 +219,7 @@ function onKeyDown(event: KeyboardEvent) {
nodeElement.value?.blur()
}
handleNodeClick(node, multiSelectionActive, addSelectedNodes, removeSelectedElements, $$(nodesSelectionActive), unselect)
handleNodeClick(node, multiSelectionActive, addSelectedNodes, removeSelectedNodes, $$(nodesSelectionActive), unselect)
} else if (!disableKeyboardA11y && draggable && node.selected && arrowKeyDiffs[event.key]) {
$$(ariaLiveMessage).value = `Moved selected node ${event.key.replace('Arrow', '').toLowerCase()}. New position, x: ${~~node
.position.x}, y: ${~~node.position.y}`

View File

@@ -28,7 +28,7 @@ export const handleNodeClick = (
node: GraphNode,
multiSelectionActive: boolean,
addSelectedNodes: Actions['addSelectedNodes'],
removeSelectedElements: Actions['removeSelectedElements'],
removeSelectedNodes: Actions['removeSelectedNodes'],
nodesSelectionActive: Ref<boolean>,
unselect = false,
) => {
@@ -37,6 +37,6 @@ export const handleNodeClick = (
if (!node.selected) {
addSelectedNodes([node])
} else if (unselect || (node.selected && multiSelectionActive)) {
removeSelectedElements([node])
removeSelectedNodes([node])
}
}