Remove tree node delete quick action

Author: beNative

components/IconButton.tsx

@@ -3,7 +3,7 @@ import Tooltip from './Tooltip';
 
 interface IconButtonProps extends React.ButtonHTMLAttributes<HTMLButtonElement> {
   children: React.ReactNode;
-  tooltip: string;
+  tooltip?: string;
   variant?: 'primary' | 'ghost' | 'destructive';
   size?: 'xs' | 'sm' | 'md';
   tooltipPosition?: 'top' | 'bottom';

components/PromptList.tsx

@@ -19,11 +19,7 @@ interface DocumentListProps {
   onImportNodes: (payload: DraggedNodeTransfer, targetId: string | null, position: 'before' | 'after' | 'inside') => void | Promise<void>;
   onDropFiles: (files: FileList, parentId: string | null) => void;
   onCopyNodeContent: (id: string) => void;
-  copyContentTooltip: string;
-  onSaveNodeToFile: (id: string) => void;
-  saveToFileTooltip: string;
   onToggleLock: (id: string, locked: boolean) => void | Promise<void>;
-  getToggleLockTooltip: (locked: boolean) => string;
   searchTerm: string;
   expandedIds: Set<string>;
   onToggleExpand: (id: string) => void;
@@ -50,11 +46,7 @@ const DocumentList: React.FC<DocumentListProps> = ({
   onImportNodes,
   onDropFiles,
   onCopyNodeContent,
-  copyContentTooltip,
-  onSaveNodeToFile,
-  saveToFileTooltip,
   onToggleLock,
-  getToggleLockTooltip,
   searchTerm,
   expandedIds,
   onToggleExpand,
@@ -264,11 +256,7 @@ const DocumentList: React.FC<DocumentListProps> = ({
                 onDropFiles={onDropFiles}
                 onToggleExpand={onToggleExpand}
                 onCopyNodeContent={onCopyNodeContent}
-                copyContentTooltip={copyContentTooltip}
                 onToggleLock={onToggleLock}
-                getToggleLockTooltip={getToggleLockTooltip}
-                onSaveNodeToFile={onSaveNodeToFile}
-                saveToFileTooltip={saveToFileTooltip}
                 searchTerm={searchTerm}
                 onMoveUp={onMoveUp}
                 onMoveDown={onMoveDown}

components/PromptTreeItem.tsx

@@ -2,7 +2,7 @@ import React, { useState, useRef, useEffect, useLayoutEffect } from 'react';
 // Fix: Correctly import the DocumentOrFolder type.
 import type { DocumentOrFolder, DraggedNodeTransfer } from '../types';
 import IconButton from './IconButton';
-import { FileIcon, FolderIcon, FolderOpenIcon, TrashIcon, ChevronRightIcon, ChevronDownIcon, CopyIcon, ArrowUpIcon, ArrowDownIcon, CodeIcon, SaveIcon, LockClosedIcon, LockOpenIcon } from './Icons';
+import { FileIcon, FolderIcon, FolderOpenIcon, ChevronRightIcon, ChevronDownIcon, CopyIcon, ArrowUpIcon, ArrowDownIcon, CodeIcon, LockClosedIcon, LockOpenIcon } from './Icons';
 import Tooltip from './Tooltip';
 
 export interface DocumentNode extends DocumentOrFolder {
@@ -30,11 +30,7 @@ interface DocumentTreeItemProps {
   onDropFiles: (files: FileList, parentId: string | null) => void;
   onToggleExpand: (id: string) => void;
   onCopyNodeContent: (id: string) => void;
-  copyContentTooltip: string;
-  onSaveNodeToFile: (id: string) => void;
-  saveToFileTooltip: string;
   onToggleLock: (id: string, locked: boolean) => void | Promise<void>;
-  getToggleLockTooltip: (locked: boolean) => string;
   isKnownNodeId: (id: string) => boolean;
   searchTerm: string;
   onMoveUp: (id: string) => void;
@@ -117,11 +113,7 @@ const DocumentTreeItem: React.FC<DocumentTreeItemProps> = (props) => {
     onDropFiles,
     onToggleExpand,
     onCopyNodeContent,
-    copyContentTooltip,
-    onSaveNodeToFile,
-    saveToFileTooltip,
     onToggleLock,
-    getToggleLockTooltip,
     onMoveUp,
     onMoveDown,
     canMoveUp,
@@ -155,6 +147,7 @@ const DocumentTreeItem: React.FC<DocumentTreeItemProps> = (props) => {
   const isOpenInTab = !isFolder && openDocumentIds.has(node.id);
   const areActionsVisible = isSelected || isFocused || isHovered;
   const emojiForNode = !isFolder ? extractEmoji(node.title) : null;
+  const lockAriaLabel = node.locked ? 'Unlock Document' : 'Lock Document';
   const displayTitle = React.useMemo(() => {
     if (!emojiForNode || isFolder) {
       return node.title;
@@ -463,23 +456,40 @@ const DocumentTreeItem: React.FC<DocumentTreeItemProps> = (props) => {
                         overflow: areActionsVisible ? undefined : 'hidden',
                     }}
                 >
-                    <IconButton onClick={(e) => { e.stopPropagation(); onMoveUp(node.id); }} tooltip="Move Up" size="xs" variant="ghost" disabled={!canMoveUp}>
+                    <IconButton
+                        aria-label="Move Up"
+                        onClick={(e) => { e.stopPropagation(); onMoveUp(node.id); }}
+                        size="xs"
+                        variant="ghost"
+                        disabled={!canMoveUp}
+                    >
                         <ArrowUpIcon className="w-3.5 h-3.5" />
                     </IconButton>
-                    <IconButton onClick={(e) => { e.stopPropagation(); onMoveDown(node.id); }} tooltip="Move Down" size="xs" variant="ghost" disabled={!canMoveDown}>
+                    <IconButton
+                        aria-label="Move Down"
+                        onClick={(e) => { e.stopPropagation(); onMoveDown(node.id); }}
+                        size="xs"
+                        variant="ghost"
+                        disabled={!canMoveDown}
+                    >
                         <ArrowDownIcon className="w-3.5 h-3.5" />
                     </IconButton>
                     {!isFolder && (
                       <>
-                        <IconButton onClick={(e) => { e.stopPropagation(); onCopyNodeContent(node.id); }} tooltip={copyContentTooltip} size="xs" variant="ghost">
+                        <IconButton
+                            aria-label="Copy Content"
+                            onClick={(e) => { e.stopPropagation(); onCopyNodeContent(node.id); }}
+                            size="xs"
+                            variant="ghost"
+                        >
                           <CopyIcon className="w-3.5 h-3.5" />
                         </IconButton>
                         <IconButton
+                          aria-label={lockAriaLabel}
                           onClick={(e) => {
                             e.stopPropagation();
                             void onToggleLock(node.id, !node.locked);
                           }}
-                          tooltip={getToggleLockTooltip(node.locked)}
                           size="xs"
                           variant="ghost"
                           className={node.locked ? 'text-primary' : ''}
@@ -490,14 +500,8 @@ const DocumentTreeItem: React.FC<DocumentTreeItemProps> = (props) => {
                             <LockOpenIcon className="w-3.5 h-3.5" />
                           )}
                         </IconButton>
-                        <IconButton onClick={(e) => { e.stopPropagation(); onSaveNodeToFile(node.id); }} tooltip={saveToFileTooltip} size="xs" variant="ghost">
-                          <SaveIcon className="w-3.5 h-3.5" />
-                        </IconButton>
                       </>
                     )}
-                    <IconButton onClick={(e) => { e.stopPropagation(); onDeleteNode(node.id, e.shiftKey); }} tooltip="Delete" size="xs" variant="destructive">
-                        <TrashIcon className="w-3.5 h-3.5" />
-                    </IconButton>
                 </div>
             )}
         </div>

components/Sidebar.tsx

@@ -510,15 +510,11 @@ const Sidebar: React.FC<SidebarProps> = (props) => {
                     onSelectNode={props.onSelectNode}
                     onDeleteNode={props.onDeleteNode}
                     onRenameNode={props.onRenameNode}
-                                        onMoveNode={props.onMoveNode}
+                    onMoveNode={props.onMoveNode}
                     onImportNodes={props.onImportNodes}
                     onDropFiles={props.onDropFiles}
-                onCopyNodeContent={props.onCopyNodeContent}
-                onToggleLock={props.onToggleNodeLock}
-                getToggleLockTooltip={(locked) => getTooltip('toggle-document-lock', locked ? 'Unlock Document' : 'Lock Document')}
-                copyContentTooltip={getTooltip('document-tree-copy-content', 'Copy Content')}
-                onSaveNodeToFile={props.onSaveNodeToFile}
-                    saveToFileTooltip={getTooltip('document-tree-save-to-file', 'Save to File')}
+                    onCopyNodeContent={props.onCopyNodeContent}
+                    onToggleLock={props.onToggleNodeLock}
                     searchTerm={searchTerm}
                     expandedIds={props.expandedFolderIds}
                     onToggleExpand={props.onToggleExpand}