Merge pull request #3426 from quantified-uncertainty/ai-remove-x

Remove close button from actions for Squiggle AI
quantified-uncertainty · Nov 8, 2024 · 5865b38 · 5865b38
2 parents aaf1b7b + af88b5c
commit 5865b38
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 11 deletions.
diff --git a/packages/hub/src/app/ai/WorkflowViewer/SelectedNodeSideView.tsx b/packages/hub/src/app/ai/WorkflowViewer/SelectedNodeSideView.tsx
@@ -2,7 +2,7 @@ import clsx from "clsx";
 import { FC, useMemo } from "react";
 
 import { ClientArtifact, ClientStep } from "@quri/squiggle-ai";
-import { ChevronLeftIcon, ChevronRightIcon, XIcon } from "@quri/ui";
+import { ChevronLeftIcon, ChevronRightIcon } from "@quri/ui";
 
 import { useAvailableHeight } from "@/hooks/useAvailableHeight";
 
@@ -34,10 +34,9 @@ const NavButton: FC<{
 
 export const SelectedNodeSideView: FC<{
   selectedNode: ClientStep;
-  onClose: () => void;
   onSelectPreviousNode?: () => void;
   onSelectNextNode?: () => void;
-}> = ({ selectedNode, onClose, onSelectPreviousNode, onSelectNextNode }) => {
+}> = ({ selectedNode, onSelectPreviousNode, onSelectNextNode }) => {
   const { ref, height } = useAvailableHeight();
 
   const selectedNodeCodeOutput = useMemo(() => {
@@ -70,13 +69,6 @@ export const SelectedNodeSideView: FC<{
             label="Next Node"
             icon={ChevronRightIcon}
           />
-          <button
-            onClick={onClose}
-            className="text-slate-400 hover:text-slate-600"
-            aria-label="Close"
-          >
-            <XIcon size={20} />
-          </button>
         </div>
       </div>
       <div className="space-y-4">

diff --git a/packages/hub/src/app/ai/WorkflowViewer/WorkflowActions.tsx b/packages/hub/src/app/ai/WorkflowViewer/WorkflowActions.tsx
@@ -72,7 +72,6 @@ export const WorkflowActions: FC<{
       {hasSelectedNode && (
         <SelectedNodeSideView
           selectedNode={workflow.steps[selectedNodeIndex]}
-          onClose={() => setSelectedNodeIndex(null)}
           onSelectPreviousNode={selectPreviousNode}
           onSelectNextNode={selectNextNode}
         />