feat: crisis overlay keyboard navigation — Escape key (fixes #95) #102

Closed
Rockachopa wants to merge 2 commits from fix/overlay-escape-key into main
2 changed files with 23 additions and 0 deletions

View File

@@ -1056,6 +1056,15 @@ Sovereignty and service always.`;
// Register focus trap on document (always listening, gated by class check)
document.addEventListener('keydown', trapFocusInOverlay);
// Escape key closes crisis overlay (returns focus to chat input)
document.addEventListener('keydown', function(e) {
if (e.key === 'Escape' && crisisOverlay.classList.contains('active')) {
if (!overlayDismissBtn.disabled) {
overlayDismissBtn.click();
}
}
});
overlayDismissBtn.addEventListener('click', function() {
if (!overlayDismissBtn.disabled) {
crisisOverlay.classList.remove('active');

View File

@@ -53,5 +53,19 @@ class TestCrisisOverlayFocusTrap(unittest.TestCase):
)
def test_overlay_escape_key_closes_overlay(self):
"""Escape key closes crisis overlay and returns focus to chat input."""
# Verify the Escape handler exists in the HTML
self.assertRegex(
self.html,
r"e\.key\s*===\s*['"]Escape['"].*crisisOverlay\.classList\.contains\('active'\)",
'Expected Escape key handler to check for active crisis overlay.',
)
self.assertRegex(
self.html,
r"overlayDismissBtn\.click\(\)",
'Expected Escape key handler to trigger overlay dismiss button click.',
)
if __name__ == '__main__':
unittest.main()