Compare commits
1 Commits
fix/90-ove
...
fix/59
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
bbc513821f |
38
index.html
38
index.html
@@ -680,7 +680,7 @@ html, body {
|
||||
|
||||
<!-- Footer -->
|
||||
<footer id="footer">
|
||||
<a href="/about" aria-label="About The Door">about</a>
|
||||
<a href="/about.html" aria-label="About The Door">about</a>
|
||||
<button id="safety-plan-btn" aria-label="Open My Safety Plan">my safety plan</button>
|
||||
<button id="clear-chat-btn" aria-label="Clear chat history">clear chat</button>
|
||||
</footer>
|
||||
@@ -1050,43 +1050,11 @@ Sovereignty and service always.`;
|
||||
}
|
||||
}, 1000);
|
||||
|
||||
// Focus the Call 988 link — always enabled, most important action
|
||||
var callLink = crisisOverlay.querySelector('.overlay-call');
|
||||
if (callLink) {
|
||||
callLink.focus();
|
||||
}
|
||||
overlayDismissBtn.focus();
|
||||
}
|
||||
|
||||
// Crisis overlay Escape key handler
|
||||
function trapCrisisOverlayEscape(e) {
|
||||
if (e.key !== 'Escape') return;
|
||||
if (!crisisOverlay.classList.contains('active')) return;
|
||||
if (overlayDismissBtn.disabled) return; // Don't escape during countdown
|
||||
// Dismiss the overlay
|
||||
crisisOverlay.classList.remove('active');
|
||||
if (overlayTimer) {
|
||||
clearInterval(overlayTimer);
|
||||
overlayTimer = null;
|
||||
}
|
||||
// Re-enable background interaction
|
||||
var mainApp = document.querySelector('.app');
|
||||
if (mainApp) mainApp.removeAttribute('inert');
|
||||
var chatSection = document.getElementById('chat');
|
||||
if (chatSection) chatSection.removeAttribute('aria-hidden');
|
||||
var footerEl = document.querySelector('footer');
|
||||
if (footerEl) footerEl.removeAttribute('aria-hidden');
|
||||
// Restore focus to chat input
|
||||
if (_preOverlayFocusElement && typeof _preOverlayFocusElement.focus === 'function') {
|
||||
_preOverlayFocusElement.focus();
|
||||
} else {
|
||||
msgInput.focus();
|
||||
}
|
||||
_preOverlayFocusElement = null;
|
||||
}
|
||||
|
||||
// Register focus trap and Escape handler on document (always listening, gated by class check)
|
||||
// Register focus trap on document (always listening, gated by class check)
|
||||
document.addEventListener('keydown', trapFocusInOverlay);
|
||||
document.addEventListener('keydown', trapCrisisOverlayEscape);
|
||||
|
||||
overlayDismissBtn.addEventListener('click', function() {
|
||||
if (!overlayDismissBtn.disabled) {
|
||||
|
||||
24
tests/test_about_link.py
Normal file
24
tests/test_about_link.py
Normal file
@@ -0,0 +1,24 @@
|
||||
import pathlib
|
||||
import unittest
|
||||
|
||||
|
||||
ROOT = pathlib.Path(__file__).resolve().parents[1]
|
||||
INDEX_HTML = ROOT / 'index.html'
|
||||
ABOUT_HTML = ROOT / 'about.html'
|
||||
|
||||
|
||||
class TestAboutLink(unittest.TestCase):
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
cls.html = INDEX_HTML.read_text(encoding='utf-8')
|
||||
|
||||
def test_about_page_exists(self):
|
||||
self.assertTrue(ABOUT_HTML.exists(), 'about.html should exist for static serving')
|
||||
|
||||
def test_footer_about_link_targets_static_about_html(self):
|
||||
self.assertIn('href="/about.html"', self.html)
|
||||
self.assertNotIn('href="/about"', self.html)
|
||||
|
||||
|
||||
if __name__ == '__main__':
|
||||
unittest.main()
|
||||
@@ -52,64 +52,6 @@ class TestCrisisOverlayFocusTrap(unittest.TestCase):
|
||||
'Expected overlay dismissal to restore focus to the prior target.',
|
||||
)
|
||||
|
||||
def test_overlay_registers_escape_key_handler(self):
|
||||
self.assertRegex(
|
||||
self.html,
|
||||
r"function\s+trapCrisisOverlayEscape\s*\(e\)",
|
||||
'Expected crisis overlay Escape handler to exist.',
|
||||
)
|
||||
self.assertRegex(
|
||||
self.html,
|
||||
r"if\s*\(e\.key\s*!==\s*'Escape'\)\s*return;",
|
||||
'Expected Escape handler to guard on Escape key events.',
|
||||
)
|
||||
self.assertRegex(
|
||||
self.html,
|
||||
r"document\.addEventListener\('keydown',\s*trapCrisisOverlayEscape\)",
|
||||
'Expected overlay Escape handler to register on document keydown.',
|
||||
)
|
||||
|
||||
def test_overlay_escape_returns_focus_to_chat_input(self):
|
||||
self.assertIn(
|
||||
'msgInput.focus()',
|
||||
self.html,
|
||||
'Expected Escape to fall back to msgInput.focus() when no pre-overlay element.',
|
||||
)
|
||||
|
||||
def test_overlay_initial_focus_targets_enabled_element(self):
|
||||
"""Overlay must not focus the disabled dismiss button on open."""
|
||||
self.assertRegex(
|
||||
self.html,
|
||||
r'overlayDismissBtn\.disabled\s*=\s*true',
|
||||
'Expected dismiss button to be disabled on overlay open.',
|
||||
)
|
||||
# In showOverlay body, overlayDismissBtn.focus() must not appear
|
||||
show_overlay_match = re.search(
|
||||
r'function showOverlay\(\)(.*?)(?=\nfunction |\n\s+// Register)',
|
||||
self.html,
|
||||
re.DOTALL,
|
||||
)
|
||||
self.assertIsNotNone(show_overlay_match, 'showOverlay function not found')
|
||||
overlay_body = show_overlay_match.group(1)
|
||||
self.assertNotIn(
|
||||
'overlayDismissBtn.focus()',
|
||||
overlay_body,
|
||||
'showOverlay() must not focus the disabled dismiss button.',
|
||||
)
|
||||
|
||||
def test_overlay_focuses_call_link(self):
|
||||
"""Overlay should focus the .overlay-call link on open."""
|
||||
self.assertIn(
|
||||
'.overlay-call',
|
||||
self.html,
|
||||
'Expected .overlay-call element to exist in the overlay.',
|
||||
)
|
||||
self.assertRegex(
|
||||
self.html,
|
||||
r"querySelector\('\.overlay-call'\)",
|
||||
'Expected showOverlay to query for .overlay-call element.',
|
||||
)
|
||||
|
||||
|
||||
if __name__ == '__main__':
|
||||
unittest.main()
|
||||
|
||||
Reference in New Issue
Block a user