feat: Improve history list keyboard navigation and bookmark form UX

- Add keyboard shortcuts for history/bookmark list:
  * Return: Open selected query in new tab
  * Cmd+C: Copy query to clipboard
  * Cmd+E: Edit bookmark (bookmarks mode only)
  * Escape: Clear search/selection
  * Delete: Delete selected entry

- Implement smart selection after deletion:
  * Automatically select next item after delete
  * Select previous if last item deleted
  * Update preview panel immediately

- Add keyboard shortcut hints to context menu items

- Improve bookmark form (AppKit):
  * Better spacing (24px margins, 16px row spacing)
  * Larger dialog (520x360)
  * Improved placeholder text
  * Larger fonts (13pt for readability)
  * Better text insets (8px)

- Fix responder chain for standard Edit menu commands

Note: Reverted from SwiftUI form due to beep sound issue with keyboard shortcuts in hybrid AppKit/SwiftUI architecture

Author: datlechin

OpenTable.xcodeproj/project.xcworkspace/xcuserdata/ngoquocdat.xcuserdatad/UserInterfaceState.xcuserstate

@@ -23,6 +23,7 @@ struct ContentView: View {
 
     @Environment(\.openWindow) private var openWindow
     @Environment(\.dismissWindow) private var dismissWindow
+    @EnvironmentObject private var appState: AppState
 
     private let storage = ConnectionStorage.shared
 

OpenTable/ContentView.swift

@@ -0,0 +1,58 @@
+//
+//  NSViewController+SwiftUI.swift
+//  OpenTable
+//
+//  Helper extension to present SwiftUI views from AppKit
+//
+
+import AppKit
+import SwiftUI
+
+extension NSViewController {
+    /// Present a SwiftUI view as a sheet with proper keyboard handling
+    func presentAsSheet<Content: View>(_ swiftUIView: Content, onSave: (() -> Void)? = nil, onCancel: (() -> Void)? = nil) {
+        let hostingController = KeyboardHandlingHostingController(rootView: swiftUIView)
+        hostingController.onSave = onSave
+        hostingController.onCancel = onCancel ?? { [weak hostingController] in
+            hostingController?.dismiss(nil)
+        }
+        presentAsSheet(hostingController)
+    }
+}
+
+/// Custom NSHostingController that properly handles keyboard events
+private class KeyboardHandlingHostingController<Content: View>: NSHostingController<Content> {
+    
+    var onSave: (() -> Void)?
+    var onCancel: (() -> Void)?
+    
+    override func performKeyEquivalent(with event: NSEvent) -> Bool {
+        // Check for Command modifier
+        let commandPressed = event.modifierFlags.contains(.command)
+        
+        // Handle Cmd+Return (Save)
+        if commandPressed && (event.keyCode == 36 || event.keyCode == 76) {
+            onSave?()
+            return true
+        }
+        
+        // Let super handle other events
+        return super.performKeyEquivalent(with: event)
+    }
+    
+    override func cancelOperation(_ sender: Any?) {
+        // Handle Escape key
+        onCancel?()
+    }
+    
+    override func keyDown(with event: NSEvent) {
+        // Check for Escape key without modifiers
+        if event.keyCode == 53 && event.modifierFlags.intersection(.deviceIndependentFlagsMask).isEmpty {
+            onCancel?()
+            return
+        }
+        
+        // Pass other keys to super
+        super.keyDown(with: event)
+    }
+}

OpenTable/Extensions/NSViewController+SwiftUI.swift

@@ -16,6 +16,7 @@ final class AppState: ObservableObject {
     @Published var isCurrentTabEditable: Bool = false  // True when current tab is an editable table
     @Published var hasRowSelection: Bool = false  // True when rows are selected in data grid
     @Published var hasTableSelection: Bool = false  // True when tables are selected in sidebar
+    @Published var isHistoryPanelVisible: Bool = false  // Global history panel visibility
 }
 
 // MARK: - App
@@ -136,6 +137,20 @@ struct OpenTableApp: App {
                 .keyboardShortcut("v", modifiers: .command)
 
                 Button("Delete") {
+                    // Check if first responder is the history panel's table view
+                    // History panel uses responder chain for delete actions
+                    // Data grid uses notifications for batched undo support
+                    if let firstResponder = NSApp.keyWindow?.firstResponder {
+                        // Check class name to identify HistoryTableView
+                        let className = String(describing: type(of: firstResponder))
+                        if className.contains("HistoryTableView") {
+                            // Let history panel handle via responder chain
+                            NSApp.sendAction(#selector(NSText.delete(_:)), to: nil, from: nil)
+                            return
+                        }
+                    }
+                    
+                    // For data grid and other views, use notification for batched undo
                     NotificationCenter.default.post(name: .deleteSelectedRows, object: nil)
                 }
                 .keyboardShortcut(.delete, modifiers: .command)

OpenTable/OpenTableApp.swift

@@ -43,8 +43,8 @@ final class BookmarkEditorController: NSViewController {
     // MARK: - Lifecycle
 
     override func loadView() {
-        // Create main container
-        let contentView = NSView(frame: NSRect(x: 0, y: 0, width: 450, height: 380))
+        // Create main container with better spacing
+        let contentView = NSView(frame: NSRect(x: 0, y: 0, width: 520, height: 360))
         contentView.translatesAutoresizingMaskIntoConstraints = false
 
         // Build form using NSGridView (native macOS form layout)
@@ -57,12 +57,12 @@ final class BookmarkEditorController: NSViewController {
         contentView.addSubview(buttonStack)
 
         NSLayoutConstraint.activate([
-            gridView.topAnchor.constraint(equalTo: contentView.topAnchor, constant: 20),
-            gridView.leadingAnchor.constraint(equalTo: contentView.leadingAnchor, constant: 20),
-            gridView.trailingAnchor.constraint(equalTo: contentView.trailingAnchor, constant: -20),
+            gridView.topAnchor.constraint(equalTo: contentView.topAnchor, constant: 24),
+            gridView.leadingAnchor.constraint(equalTo: contentView.leadingAnchor, constant: 24),
+            gridView.trailingAnchor.constraint(equalTo: contentView.trailingAnchor, constant: -24),
 
-            buttonStack.topAnchor.constraint(equalTo: gridView.bottomAnchor, constant: 20),
-            buttonStack.trailingAnchor.constraint(equalTo: contentView.trailingAnchor, constant: -20),
+            buttonStack.topAnchor.constraint(equalTo: gridView.bottomAnchor, constant: 24),
+            buttonStack.trailingAnchor.constraint(equalTo: contentView.trailingAnchor, constant: -24),
             buttonStack.bottomAnchor.constraint(equalTo: contentView.bottomAnchor, constant: -20)
         ])
 
@@ -73,7 +73,7 @@ final class BookmarkEditorController: NSViewController {
         super.viewDidLoad()
         title = isEditing ? "Edit Bookmark" : "New Bookmark"
         populateFields()
-        preferredContentSize = NSSize(width: 450, height: 400)
+        preferredContentSize = NSSize(width: 520, height: 380)
 
         // Add accessibility identifiers
         nameField.setAccessibilityIdentifier("bookmarkNameField")
@@ -91,23 +91,29 @@ final class BookmarkEditorController: NSViewController {
         // Name row
         let nameLabel = NSTextField(labelWithString: "Name:")
         nameLabel.alignment = .right
+        nameLabel.font = .systemFont(ofSize: 13)
         nameField = NSTextField()
         nameField.placeholderString = "Bookmark name"
+        nameField.font = .systemFont(ofSize: 13)
 
         // Query row
         let queryLabel = NSTextField(labelWithString: "Query:")
         queryLabel.alignment = .right
+        queryLabel.font = .systemFont(ofSize: 13)
         let queryScrollView = buildQueryScrollView()
 
         // Tags row
         let tagsLabel = NSTextField(labelWithString: "Tags:")
         tagsLabel.alignment = .right
+        tagsLabel.font = .systemFont(ofSize: 13)
         tagsField = NSTextField()
-        tagsField.placeholderString = "Optional, comma-separated"
+        tagsField.placeholderString = "e.g., reports, analytics, daily"
+        tagsField.font = .systemFont(ofSize: 13)
 
         // Notes row
         let notesLabel = NSTextField(labelWithString: "Notes:")
         notesLabel.alignment = .right
+        notesLabel.font = .systemFont(ofSize: 13)
         let notesScrollView = buildNotesScrollView()
 
         // Create grid
@@ -119,24 +125,24 @@ final class BookmarkEditorController: NSViewController {
         ])
 
         gridView.translatesAutoresizingMaskIntoConstraints = false
-        gridView.columnSpacing = 10
-        gridView.rowSpacing = 14
+        gridView.columnSpacing = 12
+        gridView.rowSpacing = 16
 
-        // Configure column widths (slightly wider for label column)
-        gridView.column(at: 0).width = 65
+        // Configure column widths - slightly wider label column
+        gridView.column(at: 0).width = 70
         gridView.column(at: 0).xPlacement = .trailing
 
-        // Configure row alignments
+        // Configure row alignments with better padding
         gridView.row(at: 0).topPadding = 0
-        gridView.row(at: 1).topPadding = 6
-        gridView.row(at: 2).topPadding = 6
-        gridView.row(at: 3).topPadding = 6
+        gridView.row(at: 1).topPadding = 8
+        gridView.row(at: 2).topPadding = 8
+        gridView.row(at: 3).topPadding = 8
 
         // Align labels to top for multi-line fields
         gridView.cell(atColumnIndex: 0, rowIndex: 1).yPlacement = .top
         gridView.cell(atColumnIndex: 0, rowIndex: 3).yPlacement = .top
 
-        // Set heights for scroll views with better proportions
+        // Set heights for scroll views
         queryScrollView.heightAnchor.constraint(equalToConstant: 90).isActive = true
         notesScrollView.heightAnchor.constraint(equalToConstant: 70).isActive = true
 
@@ -156,7 +162,7 @@ final class BookmarkEditorController: NSViewController {
         queryTextView.isSelectable = true
         queryTextView.font = .monospacedSystemFont(ofSize: 11, weight: .regular)
         queryTextView.string = query
-        queryTextView.textContainerInset = NSSize(width: 6, height: 6)
+        queryTextView.textContainerInset = NSSize(width: 8, height: 8)
         queryTextView.isVerticallyResizable = true
         queryTextView.isHorizontallyResizable = false
         queryTextView.autoresizingMask = [.width]
@@ -180,7 +186,7 @@ final class BookmarkEditorController: NSViewController {
         notesTextView = NSTextView()
         notesTextView.isRichText = false
         notesTextView.font = .systemFont(ofSize: 13)
-        notesTextView.textContainerInset = NSSize(width: 6, height: 6)
+        notesTextView.textContainerInset = NSSize(width: 8, height: 8)
         notesTextView.isVerticallyResizable = true
         notesTextView.isHorizontallyResizable = false
         notesTextView.autoresizingMask = [.width]
@@ -194,11 +200,11 @@ final class BookmarkEditorController: NSViewController {
     private func buildButtonStack() -> NSStackView {
         let cancelButton = NSButton(title: "Cancel", target: self, action: #selector(cancelAction))
         cancelButton.bezelStyle = .rounded
-        cancelButton.keyEquivalent = "\u{1b}"
+        cancelButton.keyEquivalent = "\u{1b}" // Escape
 
         let saveButton = NSButton(title: isEditing ? "Save" : "Save Bookmark", target: self, action: #selector(saveAction))
         saveButton.bezelStyle = .rounded
-        saveButton.keyEquivalent = "\r"
+        saveButton.keyEquivalent = "\r" // Return
 
         let stackView = NSStackView(views: [cancelButton, saveButton])
         stackView.orientation = .horizontal