fix: add chat-header safety plan access (#38 )

test: add chat-header safety plan regression (#38 )
2026-04-17 01:14:28 -04:00 · 2026-04-17 00:59:21 -04:00
2 changed files with 88 additions and 60 deletions
--- a/index.html
+++ b/index.html
@@ -241,6 +241,48 @@ html, body {
  opacity: 0.5;
 }

+/* ===== CHAT HEADER ===== */
+#chat-header {
+  flex-shrink: 0;
+  display: flex;
+  align-items: center;
+  justify-content: space-between;
+  gap: 12px;
+  padding: 10px 12px;
+  border-bottom: 1px solid #21262d;
+  background: #11161d;
+}
+
+.chat-header-title {
+  font-size: 0.85rem;
+  color: #8b949e;
+  font-weight: 600;
+  letter-spacing: 0.02em;
+}
+
+#chat-safety-plan-btn {
+  display: inline-flex;
+  align-items: center;
+  gap: 6px;
+  padding: 8px 12px;
+  min-height: 36px;
+  border: 1px solid #30363d;
+  border-radius: 999px;
+  background: transparent;
+  color: #c9d1d9;
+  font-size: 0.8rem;
+  font-weight: 600;
+  cursor: pointer;
+}
+
+#chat-safety-plan-btn:hover,
+#chat-safety-plan-btn:focus {
+  border-color: #58a6ff;
+  background: rgba(88, 166, 255, 0.12);
+  outline: 2px solid #58a6ff;
+  outline-offset: 2px;
+}
+
 /* ===== CHAT AREA ===== */
 #chat-area {
  flex: 1;
@@ -613,31 +655,6 @@ html, body {
  top: 8px;
  outline: 2px solid #58a6ff;
 }
-
-/* ===== TOAST NOTIFICATION (fix #73) ===== */
-.toast-notification {
-  position: fixed;
-  bottom: 24px;
-  left: 50%;
-  transform: translateX(-50%) translateY(100px);
-  padding: 12px 24px;
-  border-radius: 8px;
-  font-size: 0.9rem;
-  font-weight: 500;
-  z-index: 10001;
-  opacity: 0;
-  transition: transform 0.3s ease, opacity 0.3s ease;
-  pointer-events: none;
-  max-width: 90vw;
-  text-align: center;
-}
-.toast-notification.visible {
-  transform: translateX(-50%) translateY(0);
-  opacity: 1;
-}
-.toast-notification.success { background: #238636; color: #fff; }
-.toast-notification.error { background: #da3633; color: #fff; }
-
 </style>
 </head>
 <body>
@@ -674,6 +691,14 @@ html, body {
    </div>
  </div>

+  <div id="chat-header">
+    <div class="chat-header-title" aria-hidden="true">Conversation</div>
+    <button id="chat-safety-plan-btn" type="button" aria-label="Open My Safety Plan from chat header">
+      <svg width="16" height="16" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><path d="M14 2H6a2 2 0 0 0-2 2v16a2 2 0 0 0 2 2h12a2 2 0 0 0 2-2V8z"/><polyline points="14 2 14 8 20 8"/><line x1="16" y1="13" x2="8" y2="13"/><line x1="16" y1="17" x2="8" y2="17"/><polyline points="10 9 9 9 8 9"/></svg>
+      My Safety Plan
+    </button>
+  </div>
+
  <!-- Chat messages -->
  <div id="chat-area" role="log" aria-label="Chat messages" aria-live="polite" tabindex="0">
    <!-- Messages inserted here -->
@@ -705,7 +730,7 @@ html, body {

  <!-- Footer -->
  <footer id="footer">
-    <a href="/about.html" aria-label="About The Door">about</a>
+    <a href="/about" 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>
@@ -839,6 +864,7 @@ Sovereignty and service always.`;
  
  // Safety Plan Elements
  var safetyPlanBtn = document.getElementById('safety-plan-btn');
+  var chatSafetyPlanBtn = document.getElementById('chat-safety-plan-btn');
  var crisisSafetyPlanBtn = document.getElementById('crisis-safety-plan-btn');
  var safetyPlanModal = document.getElementById('safety-plan-modal');
  var closeSafetyPlan = document.getElementById('close-safety-plan');
@@ -1221,24 +1247,7 @@ Sovereignty and service always.`;
  });

  saveSafetyPlan.addEventListener('click', function() {
-    var 
-  // ===== TOAST NOTIFICATION (fix #73) =====
-  var _toastEl = document.getElementById('toast-notification');
-  var _toastTimer = null;
-
-  function showToast(message, type) {
-    type = type || 'success';
-    _toastEl.textContent = message;
-    _toastEl.className = 'toast-notification ' + type;
-    void _toastEl.offsetHeight; // force reflow before transition
-    _toastEl.classList.add('visible');
-    if (_toastTimer) clearTimeout(_toastTimer);
-    _toastTimer = setTimeout(function() {
-      _toastEl.classList.remove('visible');
-    }, 3000);
-  }
-
-    plan = {
+    var plan = {
      warningSigns: document.getElementById('sp-warning-signs').value,
      coping: document.getElementById('sp-coping').value,
      distraction: document.getElementById('sp-distraction').value,
@@ -1249,9 +1258,9 @@ Sovereignty and service always.`;
      localStorage.setItem('timmy_safety_plan', JSON.stringify(plan));
      safetyPlanModal.classList.remove('active');
      _restoreSafetyPlanFocus();
-      showToast('Safety plan saved.', 'success');
+      alert('Safety plan saved locally.');
    } catch (e) {
-      showToast('Error saving plan.', 'error');
+      alert('Error saving plan.');
    }
  });

@@ -1327,19 +1336,25 @@ Sovereignty and service always.`;
    _spTriggerEl = null;
  }

-  // Wire open buttons to activate focus trap
-  safetyPlanBtn.addEventListener('click', function() {
+  function openSafetyPlan(triggerEl) {
    loadSafetyPlan();
    safetyPlanModal.classList.add('active');
-    _activateSafetyPlanFocusTrap(safetyPlanBtn);
+    _activateSafetyPlanFocusTrap(triggerEl || document.activeElement);
+  }
+
+  // Wire open buttons to activate focus trap
+  safetyPlanBtn.addEventListener('click', function() {
+    openSafetyPlan(safetyPlanBtn);
+  });
+
+  chatSafetyPlanBtn.addEventListener('click', function() {
+    openSafetyPlan(chatSafetyPlanBtn);
  });

  // Crisis panel safety plan button (if crisis panel is visible)
  if (crisisSafetyPlanBtn) {
    crisisSafetyPlanBtn.addEventListener('click', function() {
-      loadSafetyPlan();
-      safetyPlanModal.classList.add('active');
-      _activateSafetyPlanFocusTrap(crisisSafetyPlanBtn);
+      openSafetyPlan(crisisSafetyPlanBtn);
    });
  }

@@ -1486,9 +1501,7 @@ Sovereignty and service always.`;
    // Check for URL params (e.g., ?safetyplan=true for PWA shortcut)
    var urlParams = new URLSearchParams(window.location.search);
    if (urlParams.get('safetyplan') === 'true') {
-      loadSafetyPlan();
-      safetyPlanModal.classList.add('active');
-      _activateSafetyPlanFocusTrap(safetyPlanBtn);
+      openSafetyPlan(chatSafetyPlanBtn || safetyPlanBtn);
      // Clean up URL
      window.history.replaceState({}, document.title, window.location.pathname);
    }
@@ -1505,10 +1518,5 @@ Sovereignty and service always.`;

 })();
 </script>
-
-  <!-- Toast notification (fix #73) -->
-  <div id="toast-notification" class="toast-notification"
-       role="status" aria-live="polite" aria-atomic="true"></div>
-
 </body>
 </html>
--- a/tests/test_safety_plan_chat_header.py
+++ b/tests/test_safety_plan_chat_header.py
@@ -0,0 +1,20 @@
+from pathlib import Path
+
+INDEX = Path("index.html")
+
+
+def test_chat_header_has_persistent_safety_plan_button():
+    html = INDEX.read_text()
+    assert 'id="chat-header"' in html
+    assert 'id="chat-safety-plan-btn"' in html
+    assert 'aria-label="Open My Safety Plan from chat header"' in html
+    assert 'My Safety Plan' in html
+
+
+def test_chat_header_button_opens_existing_safety_plan_modal():
+    html = INDEX.read_text()
+    assert "var chatSafetyPlanBtn = document.getElementById('chat-safety-plan-btn');" in html
+    assert "chatSafetyPlanBtn.addEventListener('click'" in html
+    assert "function openSafetyPlan(triggerEl)" in html
+    assert "safetyPlanModal.classList.add('active');" in html
+    assert "openSafetyPlan(chatSafetyPlanBtn);" in html
Author	SHA1	Message	Date
Alexander Whitestone	b4129bc873	fix: add chat-header safety plan access (#38 ) All checks were successful Sanity Checks / sanity-test (pull_request) Successful in 5s Details Smoke Test / smoke (pull_request) Successful in 12s Details	2026-04-17 01:14:28 -04:00
Alexander Whitestone	e3bb6b86ee	test: add chat-header safety plan regression (#38 )	2026-04-17 00:59:21 -04:00