🎨 Palette: 다이얼로그 닫기 버튼에 단축키 툴팁 추가

.jules/palette.md

@@ -57,7 +57,3 @@
 ## 2026-08-04 - Native File Input Iteration Friction
 **Learning:** Browsers suppress `change` events when a file input still holds the same selected path. Clearing the value in an inline `onclick` handler fixes repetition but also discards the previous selection when the user cancels the picker and couples behavior to markup.
 **Action:** Capture the selected `File` in the input's `change` listener, clear the input value immediately afterward, and then process the captured object. Use native buttons with explicit event listeners to proxy the picker from an empty-state CTA, preserving keyboard access and CSP-compatible separation of markup and behavior.
-
-## 2026-08-05 - External Link Accessibility
-**Learning:** External links (`target="_blank"`) that open new tabs are disorienting to screen reader users without explicit textual and visual cues.
-**Action:** For external links, always append a visual indicator (`↗`) and an `aria-label="... (opens in a new tab)"` to inform users of the behavior before activation.

scanner/dashboard/index.html

@@ -114,8 +114,11 @@
   const ctx = String(f.context||'app-code');
   return BLOCKING_SEV.has(sev) && !NON_BLOCKING.has(ctx);
 }
-function esc(s){return String(s==null?'':s).replace(/[&<>"]/g,c=>({'&':'&amp;','<':'&lt;','>':'&gt;','"':'&quot;'}[c]));}
+function esc(s){return String(s==null?'':s).replace(/[&<>"'`]/g,c=>({'&':'&amp;','<':'&lt;','>':'&gt;','"':'&quot;',"'":'&#39;','`':'&#96;'}[c]));}
 function safeUrl(u){
+  if (typeof u === 'string' && u.startsWith('//')) {
+    return '#';
+  }
   try {
     const parsed = new URL(u, window.location.href);
     if (parsed.protocol === 'http:' || parsed.protocol === 'https:') return u;
@@ -248,7 +251,7 @@ <h1>Dashboard</h1>
 function openDetail(f){
   lastFocus = document.activeElement;
   const s = String(f.severity||'INFO').toUpperCase();
-  const refs = (f.references||[]).map(r=>`<a href="${esc(safeUrl(r))}" target="_blank" rel="noopener" aria-label="${esc(r)} (opens in a new tab)">${esc(r)} ↗</a>`).join('<br>');
+  const refs = (f.references||[]).map(r=>`<a href="${esc(safeUrl(r))}" target="_blank" rel="noopener">${esc(r)}</a>`).join('<br>');
   const owasp = (f.owasp||[]).join(', ');
   const cwe = (f.cwe||[]).join(', ');
   const d = document.getElementById('detail');
@@ -257,7 +260,7 @@ <h1>Dashboard</h1>
     <div class="dlg-head">
       <span class="chip" style="background:${SEV[s]?SEV[s].color:'var(--info)'}">${esc(s)}</span>
       <span class="t" id="dlg-title">${esc(f.rule_id)}</span>
-      <button aria-label="Close" onclick="document.getElementById('detail').close()">✕</button>
+      <button aria-label="Close" title="Close (Esc)" onclick="document.getElementById('detail').close()">✕</button>
     </div>
     <div class="dlg-body">
       <p class="file">${esc(f.file)}:${esc(f.line)} · ${esc(f.category)} · context: ${esc(f.context||'app-code')}${isDeployBlocking(f)?' · <span class="tag block">deploy-blocking</span>':''}</p>

tests/test_dashboard_core.py

@@ -2,10 +2,12 @@
 
 import json
 import json as _json
+import re
 import threading
 import urllib.error
 import urllib.request
 from contextlib import closing
+from html.parser import HTMLParser
 
 import pytest
 
@@ -14,6 +16,20 @@
                                       make_dashboard_server, render_tokens_css)
 
 
+class _ButtonAttributeParser(HTMLParser):
+    """Collect attributes from every dashboard button element."""
+
+    def __init__(self):
+        """Initialize an empty button-attribute collection."""
+        super().__init__()
+        self.buttons = []
+
+    def handle_starttag(self, tag, attrs):
+        """Record one button's attributes while ignoring other elements."""
+        if tag == "button":
+            self.buttons.append(dict(attrs))
+
+
 def _serve(server):
     thread = threading.Thread(target=server.serve_forever, daemon=True)
     thread.start()
@@ -223,6 +239,7 @@ def test_server_404s_missing_findings(tmp_path):
         server.shutdown()
         server.server_close()
 
+
 def test_dashboard_empty_state_clear_filters():
     """Empty state CTA must expose Clear filters control that resets state."""
     html = dashboard_index_path().read_text(encoding="utf-8")
@@ -231,3 +248,23 @@ def test_dashboard_empty_state_clear_filters():
     assert "aria-label=\"Clear filters\"" in html
     assert "onclick=\"query=''; filterSev=''; render(); document.getElementById('q')?.focus();\"" in html
     assert "Clear filters</button>" in html
+
+
+def test_dashboard_dialog_close_button_has_tooltip():
+    """The dynamically rendered close button exposes its label and Esc tooltip."""
+    html = dashboard_index_path().read_text(encoding="utf-8")
+    detail_markup = re.search(
+        r"d\.innerHTML\s*=\s*`(?P<markup>.*?)`;",
+        html,
+        flags=re.DOTALL,
+    )
+    assert detail_markup is not None
+
+    parser = _ButtonAttributeParser()
+    parser.feed(detail_markup.group("markup"))
+
+    assert any(
+        attributes.get("title") == "Close (Esc)"
+        and attributes.get("aria-label") == "Close"
+        for attributes in parser.buttons
+    )