feat: add Ctrl+F, Ctrl+H, F3 keyboard shortcuts

Wires three QShortcut objects (ApplicationShortcut context) in
_connect_signals: Ctrl+F and Ctrl+H switch to the Find/Replace tab
and focus the respective field; F3 triggers _on_find_clicked.
Adds _focus_find_edit/_focus_replace_edit helpers and caches
_tab_widget + _find_replace_tab_index at load time.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: chrisdpurcell

src/views/main_window.py

@@ -16,8 +16,8 @@
 import os
 from typing import TypeVar, cast
 
-from PySide6.QtCore import QDir, QFile, QModelIndex
-from PySide6.QtGui import QAction
+from PySide6.QtCore import QDir, QFile, QModelIndex, Qt
+from PySide6.QtGui import QAction, QKeySequence, QShortcut
 from PySide6.QtUiTools import QUiLoader
 from PySide6.QtWidgets import (
     QApplication,
@@ -30,6 +30,7 @@
     QMessageBox,
     QPlainTextEdit,
     QPushButton,
+    QTabWidget,
     QTreeView,
 )
 
@@ -161,6 +162,20 @@ def _load_ui(self) -> None:
         self._action_preferences = _require(
             self.ui.findChild(QAction, "actionPreferences"), "actionPreferences"
         )
+        # Tab widget — needed by focus-navigation shortcuts to reveal Find/Replace tab.
+        self._tab_widget = _require(
+            self.ui.findChild(QTabWidget, "tabWidget"), "tabWidget"
+        )
+        # Find/Replace is the last tab; cache its index so shortcuts stay correct
+        # if tabs are reordered. Falls back to -1 (no-op) if tab is not found.
+        self._find_replace_tab_index = next(
+            (
+                i
+                for i in range(self._tab_widget.count())
+                if self._tab_widget.tabText(i).lower().startswith("find")
+            ),
+            -1,
+        )
 
     def _setup_file_tree(self) -> None:
         """Configure QFileSystemModel rooted at the user's home directory."""
@@ -224,8 +239,37 @@ def _connect_signals(self) -> None:
             lambda _: self._update_title()
         )
 
+        # Keyboard shortcuts not present in the .ui file.
+        # (Ctrl+S/O/Q/Shift+S are already wired via QAction shortcuts in main_window.ui.)
+        # ApplicationShortcut context: fires even when a child widget holds focus,
+        # rather than requiring the window itself to be active. Necessary for focus
+        # navigation shortcuts that must work regardless of which widget is focused.
+        ctrl_f = QShortcut(QKeySequence("Ctrl+F"), self.ui)
+        ctrl_f.setContext(Qt.ShortcutContext.ApplicationShortcut)
+        ctrl_f.activated.connect(self._focus_find_edit)
+
+        ctrl_h = QShortcut(QKeySequence("Ctrl+H"), self.ui)
+        ctrl_h.setContext(Qt.ShortcutContext.ApplicationShortcut)
+        ctrl_h.activated.connect(self._focus_replace_edit)
+
+        f3 = QShortcut(QKeySequence("F3"), self.ui)
+        f3.setContext(Qt.ShortcutContext.ApplicationShortcut)
+        f3.activated.connect(self._on_find_clicked)
+
     # ---------------------------------------------------------- user actions
 
+    def _focus_find_edit(self) -> None:
+        """Switch to Find/Replace tab and focus the find field (Ctrl+F target)."""
+        if self._find_replace_tab_index >= 0:
+            self._tab_widget.setCurrentIndex(self._find_replace_tab_index)
+        self._find_edit.setFocus()
+
+    def _focus_replace_edit(self) -> None:
+        """Switch to Find/Replace tab and focus the replace field (Ctrl+H target)."""
+        if self._find_replace_tab_index >= 0:
+            self._tab_widget.setCurrentIndex(self._find_replace_tab_index)
+        self._replace_edit.setFocus()
+
     def _on_tree_item_clicked(self, index: QModelIndex) -> None:
         """Load file on tree click; ignore directory clicks."""
         path = self._fs_model.filePath(index)

tests/integration/test_main_window.py

@@ -344,3 +344,64 @@ def test_opens_on_clean_tab(self, window):
         tab = window.ui.findChild(QTabWidget, "tabWidget")
         assert tab is not None
         assert tab.currentIndex() == 0
+
+
+class TestKeyboardShortcuts:
+    """Verify QShortcut wiring by activating each shortcut's signal directly.
+
+    qtbot.keyClick sends QKeyEvent, which does not trigger QShortcut (those
+    listen for QShortcutEvent dispatched by Qt's shortcut system). Emitting
+    activated directly tests the actual signal-to-slot connection without
+    depending on OS-level key event routing.
+
+    hasFocus() is unreliable in headless tests because it requires the window
+    to hold true OS-level focus. Instead: for Ctrl+F and Ctrl+H, the observable
+    side effect is the tab switching to Find/Replace — that is what we assert.
+    For F3, the cursor position advancing to the second match is the assertion.
+    """
+
+    def _shortcut_for(self, window, key_string: str):
+        """Return the QShortcut whose key matches key_string, or raise."""
+        from PySide6.QtGui import QKeySequence, QShortcut
+        target = QKeySequence(key_string)
+        shortcuts = window.ui.findChildren(QShortcut)
+        for sc in shortcuts:
+            if sc.key() == target:
+                return sc
+        raise AssertionError(f"No QShortcut with key '{key_string}' found on window.ui")
+
+    def test_ctrl_f_switches_to_find_replace_tab(self, window, qtbot):
+        """Ctrl+F switches to the Find/Replace tab (tab index matches _find_replace_tab_index)."""
+        from PySide6.QtWidgets import QTabWidget
+        tab = window.ui.findChild(QTabWidget, "tabWidget")
+        assert tab.currentIndex() == 0, "precondition: starts on Clean tab"
+        sc = self._shortcut_for(window, "Ctrl+F")
+        sc.activated.emit()
+        qtbot.wait(10)
+        assert tab.currentIndex() == window._find_replace_tab_index
+
+    def test_ctrl_h_switches_to_find_replace_tab(self, window, qtbot):
+        """Ctrl+H switches to the Find/Replace tab (tab index matches _find_replace_tab_index)."""
+        from PySide6.QtWidgets import QTabWidget
+        tab = window.ui.findChild(QTabWidget, "tabWidget")
+        assert tab.currentIndex() == 0, "precondition: starts on Clean tab"
+        sc = self._shortcut_for(window, "Ctrl+H")
+        sc.activated.emit()
+        qtbot.wait(10)
+        assert tab.currentIndex() == window._find_replace_tab_index
+
+    def test_f3_triggers_find_next(self, window, qtbot):
+        """F3 finds the next occurrence of the current search term."""
+        window._plain_text_edit.setPlainText("hello world hello")
+        window._find_edit.setText("hello")
+        # First find via direct call establishes an initial selection
+        window._on_find_clicked()
+        assert window._plain_text_edit.textCursor().selectedText() == "hello"
+        pos_after_first = window._plain_text_edit.textCursor().position()
+        # F3 shortcut should advance to next occurrence
+        sc = self._shortcut_for(window, "F3")
+        sc.activated.emit()
+        qtbot.wait(10)
+        assert window._plain_text_edit.textCursor().selectedText() == "hello"
+        pos_after_second = window._plain_text_edit.textCursor().position()
+        assert pos_after_second > pos_after_first