Skip to content

Commit 8652630

Browse files
committed
feat: update FlowPanelControlComponent to improve node selection logic and tooltip behavior
1 parent 6621db0 commit 8652630

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

src/packages/ce/src/flow/components/panels/FlowPanelControlComponent.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ export const FlowPanelControlComponent: React.FC<FlowPanelControlComponentProps>
6363
}, [flowId, flowService, flowStore, selectedNode])
6464

6565
useHotkeys('shift+a', (keyboardEvent) => {
66-
if (selectedNode && selectedNode.data.nodeId) setSuggestionDialogOpen(true)
66+
if (selectedNode && !selectedNode.data.functionId) setSuggestionDialogOpen(true)
6767
else setAddNextNodeTooltipOpen(true)
6868
keyboardEvent.stopPropagation()
6969
keyboardEvent.preventDefault()
@@ -97,11 +97,11 @@ export const FlowPanelControlComponent: React.FC<FlowPanelControlComponentProps>
9797
<Tooltip open={addNextNodeTooltipOpen} onOpenChange={setAddNextNodeTooltipOpen}>
9898
<TooltipTrigger asChild>
9999
<Button data-qa-selector={"flow-builder-control-panel-add"}
100-
disabled={!selectedNode || !selectedNode.data.nodeId}
100+
disabled={!selectedNode || !!selectedNode.data.functionId}
101101
paddingSize={"xxs"}
102102
variant={"filled"}
103103
onClick={() => {
104-
if (selectedNode && selectedNode.data.nodeId) setSuggestionDialogOpen(true)
104+
if (selectedNode && !selectedNode.data.functionId) setSuggestionDialogOpen(true)
105105
}}
106106
color={"secondary"}>
107107
<Text display={"flex"} align={"center"} style={{gap: "0.35rem"}}>
@@ -111,7 +111,7 @@ export const FlowPanelControlComponent: React.FC<FlowPanelControlComponentProps>
111111
</Button>
112112
</TooltipTrigger>
113113
<TooltipPortal>
114-
{!selectedNode || !selectedNode.data.nodeId && <TooltipContent sideOffset={8}>
114+
{!selectedNode || !!selectedNode.data.functionId && <TooltipContent sideOffset={8}>
115115
<Text>Select a node to add a next node</Text>
116116
</TooltipContent>}
117117
</TooltipPortal>

0 commit comments

Comments
 (0)