fix(a11y): add keyboard shortcuts, accessible descriptions, and wcag contrast to find-replace dialog

Author: lukisch

tests/test_ui_ux_accessibility.py

@@ -3,6 +3,8 @@
 """Unit- und Contract-Tests für UI-, UX-, Accessibility- und Sprach-Qualität in CodeBox."""
 
 import pytest
+from PySide6.QtCore import Qt, QEvent
+from PySide6.QtGui import QKeyEvent
 from PySide6.QtWidgets import QApplication
 from unittest.mock import patch
 
@@ -11,6 +13,7 @@
 from ui.shortcuts_dialog import ShortcutsDialog, SHORTCUTS_DATA
 from ui.plugins_dialog import PluginsDialog
 from ui.problems_panel import ProblemsPanel
+from ui.search_dialog import FindReplaceDialog
 from core.tabs import TabWidget
 from core.output import OutputPanel
 
@@ -171,3 +174,99 @@ def test_problems_and_output_panel_a11y(qapp):
     assert output.stop_btn.accessibleName() == "Ausführung stoppen"
     assert output.clear_btn.accessibleName() == "Ausgabe leeren"
     assert output.output.accessibleName() == "Programmausgabe"
+
+
+def test_find_replace_dialog_ux_and_a11y(qapp):
+    """Prüft Barrierefreiheit, Buddies, Tastenkürzel, Tab-Reihenfolge und Kontrast des Suchdialogs."""
+    dialog = FindReplaceDialog(None)
+    try:
+        # Buddies für Screenreader & mnemonics
+        assert dialog.search_label.buddy() == dialog.search_input
+        assert dialog.replace_label.buddy() == dialog.replace_input
+
+        # Accessible Attributes & Tooltips
+        assert dialog.search_input.accessibleName() == "Suchtext"
+        assert len(dialog.search_input.accessibleDescription()) > 10
+        assert "Shift+Enter" in dialog.search_input.toolTip()
+
+        assert dialog.btn_prev.accessibleName() == "Vorheriger Treffer"
+        assert len(dialog.btn_prev.accessibleDescription()) > 10
+
+        assert dialog.btn_next.accessibleName() == "Nächster Treffer"
+        assert len(dialog.btn_next.accessibleDescription()) > 10
+
+        assert dialog.replace_input.accessibleName() == "Ersetzungstext"
+        assert len(dialog.replace_input.accessibleDescription()) > 10
+
+        assert dialog.btn_replace.accessibleName() == "Treffer ersetzen"
+        assert len(dialog.btn_replace.accessibleDescription()) > 10
+        assert dialog.btn_replace.shortcut().toString() == "Alt+R"
+
+        assert dialog.btn_replace_all.accessibleName() == "Alle Treffer ersetzen"
+        assert len(dialog.btn_replace_all.accessibleDescription()) > 10
+        assert dialog.btn_replace_all.shortcut().toString() == "Alt+A"
+
+        assert dialog.cb_case.accessibleName() == "Groß- und Kleinschreibung beachten"
+        assert len(dialog.cb_case.accessibleDescription()) > 10
+
+        assert dialog.cb_words.accessibleName() == "Nur ganze Wörter suchen"
+        assert len(dialog.cb_words.accessibleDescription()) > 10
+
+        assert dialog.cb_regex.accessibleName() == "Regulären Ausdruck verwenden"
+        assert len(dialog.cb_regex.accessibleDescription()) > 10
+
+        assert dialog.status_label.accessibleName() == "Suchstatus"
+        assert len(dialog.status_label.accessibleDescription()) > 10
+
+        assert dialog.btn_toggle_preview.accessibleName() == "Voransicht umschalten"
+        assert len(dialog.btn_toggle_preview.accessibleDescription()) > 10
+        assert dialog.btn_toggle_preview.shortcut().toString() == "Alt+V"
+
+        assert dialog.preview_table.accessibleName() == "Voransichtstabelle"
+        assert len(dialog.preview_table.accessibleDescription()) > 10
+
+        assert dialog.btn_close.accessibleName() == "Dialog schließen"
+        assert len(dialog.btn_close.accessibleDescription()) > 10
+    finally:
+        dialog.close()
+
+
+def test_find_replace_keyboard_and_focus_a11y(qapp):
+    """Prüft Shift+Enter im Suchfeld, itemActivated auf Tabelle und Fokus-Rückgabe an den Editor."""
+    window = MainWindow()
+    window.new_file()
+    tab = window.tab_widget.current_tab()
+    tab.editor.setPlainText("item_alpha\nitem_beta\nitem_alpha")
+
+    dialog = FindReplaceDialog(window, initial_mode="find")
+    dialog.show()
+    qapp.processEvents()
+
+    try:
+        # Suchen nach item_alpha
+        dialog.search_input.setText("item_alpha")
+        dialog._update_live_results()
+        assert dialog.preview_table.rowCount() == 2
+
+        # Tabelle: itemActivated (Tastatur-Enter auf Element) springt zum Treffer
+        item = dialog.preview_table.item(1, 0)
+        dialog.preview_table.itemActivated.emit(item)
+        assert tab.editor.textCursor().selectedText() == "item_alpha"
+        assert tab.editor.textCursor().selectionStart() == 21
+
+        # Shift+Enter im Suchfeld löst find_prev aus
+        shift_enter = QKeyEvent(
+            QEvent.Type.KeyPress,
+            Qt.Key.Key_Return,
+            Qt.KeyboardModifier.ShiftModifier,
+        )
+        handled = dialog.eventFilter(dialog.search_input, shift_enter)
+        assert handled is True
+        # Sprang rückwärts zum ersten Treffer bei Position 0
+        assert tab.editor.textCursor().selectionStart() == 0
+
+        # Schließen des Dialogs gibt Fokus an den Editor zurück
+        dialog.close()
+        assert not dialog.isVisible()
+    finally:
+        window.close()

ui/search_dialog.py

@@ -10,6 +10,7 @@
 from typing import TYPE_CHECKING, Optional
 
 from PySide6.QtCore import Qt, QTimer, Signal
+from PySide6.QtGui import QKeySequence
 from PySide6.QtWidgets import (
     QCheckBox,
     QDialog,
@@ -62,27 +63,32 @@ def _setup_ui(self):
         grid.setSpacing(6)
 
         # Suchen
-        self.search_label = QLabel("Suchen nach:")
+        self.search_label = QLabel("&Suchen nach:")
+        self.search_label.setToolTip("Suchfeld für Text oder regulären Ausdruck (Alt+S)")
         self.search_input = QLineEdit()
         self.search_input.setObjectName("find_search_input")
         self.search_input.setPlaceholderText("Suchbegriff oder Regex...")
         self.search_input.setClearButtonEnabled(True)
-        self.search_input.setToolTip("Suchbegriff oder regulären Ausdruck eingeben (Enter für nächsten Treffer)")
+        self.search_input.setToolTip("Suchbegriff oder regulären Ausdruck eingeben (Enter für nächsten, Shift+Enter für vorherigen Treffer)")
         self.search_input.setAccessibleName("Suchtext")
         self.search_input.setAccessibleDescription("Eingabefeld für den zu suchenden Text oder regulären Ausdruck")
+        self.search_label.setBuddy(self.search_input)
+        self.search_input.installEventFilter(self)
         self.search_input.textChanged.connect(self._schedule_update)
         self.search_input.returnPressed.connect(self.find_next)
 
         self.btn_prev = QPushButton("↑ Vorheriger")
         self.btn_prev.setObjectName("find_prev_btn")
         self.btn_prev.setToolTip("Vorherigen Treffer anspringen (Shift+Enter / Shift+F3)")
         self.btn_prev.setAccessibleName("Vorheriger Treffer")
+        self.btn_prev.setAccessibleDescription("Springt zum vorherigen Treffer im Dokument (Shift+Enter oder Shift+F3)")
         self.btn_prev.clicked.connect(self.find_prev)
 
         self.btn_next = QPushButton("↓ Nächster")
         self.btn_next.setObjectName("find_next_btn")
         self.btn_next.setToolTip("Nächsten Treffer anspringen (Enter / F3)")
         self.btn_next.setAccessibleName("Nächster Treffer")
+        self.btn_next.setAccessibleDescription("Springt zum nächsten Treffer im Dokument (Enter oder F3)")
         self.btn_next.clicked.connect(self.find_next)
 
         grid.addWidget(self.search_label, 0, 0)
@@ -91,27 +97,33 @@ def _setup_ui(self):
         grid.addWidget(self.btn_next, 0, 3)
 
         # Ersetzen
-        self.replace_label = QLabel("Ersetzen durch:")
+        self.replace_label = QLabel("Erset&zen durch:")
+        self.replace_label.setToolTip("Eingabefeld für den Ersetzungstext (Alt+Z)")
         self.replace_input = QLineEdit()
         self.replace_input.setObjectName("find_replace_input")
         self.replace_input.setPlaceholderText("Ersetzungstext (unterstützt Regex-Gruppen \\1, \\2)...")
         self.replace_input.setClearButtonEnabled(True)
         self.replace_input.setToolTip("Ersetzungstext eingeben (bei Regex: \\1, \\2 für Capture-Groups)")
         self.replace_input.setAccessibleName("Ersetzungstext")
         self.replace_input.setAccessibleDescription("Eingabefeld für den Text, durch den Treffer ersetzt werden")
+        self.replace_label.setBuddy(self.replace_input)
         self.replace_input.textChanged.connect(self._schedule_update)
         self.replace_input.returnPressed.connect(self.replace_current)
 
         self.btn_replace = QPushButton("Ersetzen")
         self.btn_replace.setObjectName("replace_btn")
+        self.btn_replace.setShortcut(QKeySequence("Alt+R"))
         self.btn_replace.setToolTip("Aktuellen Treffer ersetzen (Alt+R)")
         self.btn_replace.setAccessibleName("Treffer ersetzen")
+        self.btn_replace.setAccessibleDescription("Ersetzt das aktuell ausgewählte Vorkommen durch den Ersetzungstext (Alt+R)")
         self.btn_replace.clicked.connect(self.replace_current)
 
         self.btn_replace_all = QPushButton("Alle ersetzen")
         self.btn_replace_all.setObjectName("replace_all_btn")
+        self.btn_replace_all.setShortcut(QKeySequence("Alt+A"))
         self.btn_replace_all.setToolTip("Alle Treffer im Dokument auf einmal ersetzen (Alt+A)")
         self.btn_replace_all.setAccessibleName("Alle Treffer ersetzen")
+        self.btn_replace_all.setAccessibleDescription("Ersetzt alle Vorkommen im gesamten Dokument durch den Ersetzungstext (Alt+A)")
         self.btn_replace_all.clicked.connect(self.replace_all)
 
         grid.addWidget(self.replace_label, 1, 0)
@@ -127,22 +139,25 @@ def _setup_ui(self):
 
         self.cb_case = QCheckBox("Groß-/&Kleinschreibung")
         self.cb_case.setObjectName("find_case_checkbox")
-        self.cb_case.setToolTip("Groß- und Kleinschreibung exakt beachten")
+        self.cb_case.setToolTip("Groß- und Kleinschreibung exakt beachten (Alt+K)")
         self.cb_case.setAccessibleName("Groß- und Kleinschreibung beachten")
+        self.cb_case.setAccessibleDescription("Unterscheidet bei der Suche strikt zwischen Groß- und Kleinschreibung (Alt+K)")
         self.cb_case.stateChanged.connect(self._schedule_update)
         opt_layout.addWidget(self.cb_case)
 
         self.cb_words = QCheckBox("Ganzes &Wort")
         self.cb_words.setObjectName("find_words_checkbox")
-        self.cb_words.setToolTip("Nur ganze Wörter suchen (Wortgrenzen beachten)")
+        self.cb_words.setToolTip("Nur ganze Wörter suchen (Wortgrenzen beachten, Alt+W)")
         self.cb_words.setAccessibleName("Nur ganze Wörter suchen")
+        self.cb_words.setAccessibleDescription("Findet nur Vorkommen, die als eigenständige ganze Wörter vorliegen (Alt+W)")
         self.cb_words.stateChanged.connect(self._schedule_update)
         opt_layout.addWidget(self.cb_words)
 
         self.cb_regex = QCheckBox("Re&gex")
         self.cb_regex.setObjectName("find_regex_checkbox")
-        self.cb_regex.setToolTip("Suchbegriff als regulären Python-Ausdruck interpretieren")
+        self.cb_regex.setToolTip("Suchbegriff als regulären Python-Ausdruck interpretieren (Alt+G)")
         self.cb_regex.setAccessibleName("Regulären Ausdruck verwenden")
+        self.cb_regex.setAccessibleDescription("Interpretiert den Suchbegriff als regulären Python-Ausdruck mit Gruppen-Unterstützung (Alt+G)")
         self.cb_regex.stateChanged.connect(self._schedule_update)
         opt_layout.addWidget(self.cb_regex)
 
@@ -154,15 +169,18 @@ def _setup_ui(self):
         self.status_label = QLabel("Bereit")
         self.status_label.setObjectName("find_status_label")
         self.status_label.setAccessibleName("Suchstatus")
+        self.status_label.setAccessibleDescription("Zeigt die Anzahl der gefundenen Treffer oder Syntaxfehler an")
         status_bar_layout.addWidget(self.status_label)
         status_bar_layout.addStretch()
 
         self.btn_toggle_preview = QPushButton("Voransicht")
         self.btn_toggle_preview.setObjectName("toggle_preview_btn")
         self.btn_toggle_preview.setCheckable(True)
         self.btn_toggle_preview.setChecked(True)
-        self.btn_toggle_preview.setToolTip("Voransicht der Treffer und Ersetzungen ein-/ausblenden")
+        self.btn_toggle_preview.setShortcut(QKeySequence("Alt+V"))
+        self.btn_toggle_preview.setToolTip("Voransicht der Treffer und Ersetzungen ein-/ausblenden (Alt+V)")
         self.btn_toggle_preview.setAccessibleName("Voransicht umschalten")
+        self.btn_toggle_preview.setAccessibleDescription("Schaltet die tabellarische Voransicht aller Ersetzungen ein oder aus (Alt+V)")
         self.btn_toggle_preview.toggled.connect(self._on_toggle_preview)
         status_bar_layout.addWidget(self.btn_toggle_preview)
         root_layout.addLayout(status_bar_layout)
@@ -185,9 +203,10 @@ def _setup_ui(self):
         self.preview_table.setSelectionMode(QTableWidget.SelectionMode.SingleSelection)
         self.preview_table.setEditTriggers(QTableWidget.EditTrigger.NoEditTriggers)
         self.preview_table.setAccessibleName("Voransichtstabelle")
-        self.preview_table.setAccessibleDescription("Tabelle der gefundenen Treffer mit Zeile, Spalte und Vorschau der Ersetzung")
+        self.preview_table.setAccessibleDescription("Tabelle der gefundenen Treffer mit Zeile, Spalte und Vorschau der Ersetzung (Enter auf Zeile springt zum Treffer)")
         self.preview_table.itemClicked.connect(self._on_table_item_activated)
         self.preview_table.itemDoubleClicked.connect(self._on_table_item_activated)
+        self.preview_table.itemActivated.connect(self._on_table_item_activated)
         preview_layout.addWidget(self.preview_table)
 
         root_layout.addWidget(self.preview_group, 1)
@@ -199,10 +218,24 @@ def _setup_ui(self):
         self.btn_close.setObjectName("find_close_btn")
         self.btn_close.setToolTip("Suchdialog schließen (Esc)")
         self.btn_close.setAccessibleName("Dialog schließen")
+        self.btn_close.setAccessibleDescription("Schließt den Suchen- und Ersetzen-Dialog und kehrt zum Editor zurück (Esc)")
         self.btn_close.clicked.connect(self.close)
         bottom_layout.addWidget(self.btn_close)
         root_layout.addLayout(bottom_layout)
 
+        # Logische Tab-Reihenfolge (Barrierefreie Tastaturnavigation)
+        self.setTabOrder(self.search_input, self.btn_prev)
+        self.setTabOrder(self.btn_prev, self.btn_next)
+        self.setTabOrder(self.btn_next, self.replace_input)
+        self.setTabOrder(self.replace_input, self.btn_replace)
+        self.setTabOrder(self.btn_replace, self.btn_replace_all)
+        self.setTabOrder(self.btn_replace_all, self.cb_case)
+        self.setTabOrder(self.cb_case, self.cb_words)
+        self.setTabOrder(self.cb_words, self.cb_regex)
+        self.setTabOrder(self.cb_regex, self.btn_toggle_preview)
+        self.setTabOrder(self.btn_toggle_preview, self.preview_table)
+        self.setTabOrder(self.preview_table, self.btn_close)
+
     def _get_current_editor(self) -> Optional["CodeEditor"]:
         if not self.main_window or not hasattr(self.main_window, "tab_widget"):
             return None
@@ -266,7 +299,10 @@ def _update_live_results(self):
         if err:
             editor.clearSearchHighlight()
             self.status_label.setText(f"Regex-Fehler: {err}")
-            self.status_label.setStyleSheet("color: #ff6b6b; font-weight: bold;")
+            # Barrierefreie Farbweiche für WCAG AA Kontrast (>= 4.5:1)
+            is_dark = self.palette().color(self.backgroundRole()).lightness() < 128
+            err_color = "#ff7b72" if is_dark else "#b91c1c"
+            self.status_label.setStyleSheet(f"color: {err_color}; font-weight: bold;")
             self.preview_table.setRowCount(0)
             return
 
@@ -397,6 +433,14 @@ def replace_all(self):
             self.main_window.status_bar.showMessage(f"{count} Vorkommen ersetzt.", 4000)
         self._update_live_results()
 
+    def eventFilter(self, watched, event):
+        if watched == self.search_input and event.type() == event.Type.KeyPress:
+            if event.key() in (Qt.Key.Key_Return, Qt.Key.Key_Enter):
+                if event.modifiers() & Qt.KeyboardModifier.ShiftModifier:
+                    self.find_prev()
+                    return True
+        return super().eventFilter(watched, event)
+
     def keyPressEvent(self, event):
         if event.key() == Qt.Key.Key_Escape:
             self.close()
@@ -413,4 +457,5 @@ def closeEvent(self, event):
         editor = self._get_current_editor()
         if editor:
             editor.clearSearchHighlight()
+            editor.setFocus()
         event.accept()

ui/shortcuts_dialog.py

@@ -34,6 +34,9 @@
     ("Bearbeiten", "Ersetzen", "Ctrl+H", "Öffnet den Suchen- und Ersetzen-Dialog im Ersetzen-Modus"),
     ("Bearbeiten", "Weitersuchen", "F3", "Springt zum nächsten Suchtreffer"),
     ("Bearbeiten", "Rückwärts suchen", "Shift+F3", "Springt zum vorherigen Suchtreffer"),
+    ("Bearbeiten", "Treffer ersetzen (Dialog)", "Alt+R", "Ersetzt das aktuell ausgewählte Vorkommen"),
+    ("Bearbeiten", "Alle ersetzen (Dialog)", "Alt+A", "Ersetzt alle Vorkommen im gesamten Dokument"),
+    ("Bearbeiten", "Vorheriger Treffer (Suchfeld)", "Shift+Enter", "Springt aus dem Suchfeld zum vorherigen Treffer"),
     ("Bearbeiten", "Gehe zu Zeile", "Ctrl+G", "Springt zu einer bestimmten Zeilennummer"),
     ("Bearbeiten", "Einstellungen", "Ctrl+,", "Öffnet die Programmeinstellungen"),
     ("Bearbeiten", "Plugins & Sprachen", "Ctrl+Shift+P", "Öffnet die Plugin- und Sprachverwaltung"),