fix: quarantine enqueue, dark calendar icons, color variable fixes#73
Merged
fix: quarantine enqueue, dark calendar icons, color variable fixes#73
Conversation
Quarantine: - Gateway now enqueues quarantined tool calls to quarantine_queue (was only logging status without enqueue) - Hooks handler now enqueues quarantined tool calls - HookStore interface extended with QuarantineManager - This fixes empty Quarantine tab in Events page Calendar icons: - Add color-scheme:dark to :root so native date inputs render with light icons on dark background Color variables: - Fix --warning references to --warn (the defined variable) - Fix --info references to #58a6ff (GitHub blue, no var defined)
Calendar: webkit-calendar-picker-indicator filter:invert(1) makes the native date picker icon white on dark background. Session trace: stats grid cells centered (text-align:center). AI analysis panel: strong uses --text, links use --accent-light for consistent GitHub dark mode colors.
Global CSS variables updated to full GitHub Primer dark mode spec: - Canvas: --bg, --surface, --surface2, --border, --border-muted - Text: --text, --text2, --text3, --text-on-emphasis (WCAG AA) - Semantic: danger/success/warn each with -fg/-emphasis/-muted/-border - Purple: for agent identity and sessions - Accent: #58a6ff (GitHub blue) replacing #6366f1 Badge system migrated to semi-transparent bg + solid text + border: - .badge-delivered: success-muted bg, success text, success-border - .badge-blocked: danger-muted bg, danger text, danger-border - .badge-quarantined: warn-muted bg, warn text, warn-border - All badges use 12px border-radius (pill shape) Session trace: - Timeline dots with box-shadow glow per status - Blocked/quarantined steps with tinted backgrounds - Human role badge uses purple tokens - Verdict badges use semantic muted/border pattern Security Posture: - Severity badges use semantic variables Sessions page: - Risk pills and threat badges use semantic variables Calendar icons: - WebKit picker indicator inverted for dark mode visibility
Complete color system migration across entire dashboard: Killed colors (0 remaining): - #fb923c (wrong orange for HIGH) -> var(--danger) #f85149 - #ef4444, #e06060 (wrong reds) -> #f85149 - #22c55e, #4ade80 (wrong greens) -> #3fb950 - #6366f1, #818cf8, #4f46e5 (old purple accent) -> #58a6ff/#1f6feb - #a78bfa (old violet) -> #bc8cff - #60a5fa (old blue) -> #58a6ff - All hardcoded rgba patterns -> CSS variables CSS variables: full semantic system with -muted/-border/-emphasis for danger, success, warn, accent, purple. Cache-buster: dashboard.css?v=20260320 prevents stale CSS. Badge system: all badges now use muted bg + solid text + border. Links: all #58a6ff. Purple: all #bc8cff. Agent graph colors updated.
Login page: - Remove backdrop gradient glow (flat #0d1117 background) - Card border: #30363d (was rgba white tint) - Input border: #30363d, focus: #58a6ff with blue glow (was purple) - Authenticate button: #1f6feb blue (was purple #2d2b55) - Code spans: #21262d bg with #30363d border Sidebar navigation: - Active item: #e6edf3 text, rgba(56,139,253,0.15) bg, #58a6ff left border - Inactive: #8b949e text - Hover: rgba(139,148,158,0.08) - Section labels: #6e7681, 11px Re-analyze button: - #1f6feb bg with accent-border, 6px radius (matches CSV/SARIF/JSON) - Hover: #388bfd Global cleanup: - Replace all rgba(99,102,241,...) purple glows with rgba(56,139,253,...) - Fix .btn hover from #5b52f0 to #388bfd - Fix tmpl_audit.go and tmpl_alerts.go remaining purple rgba
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Three fixes:
Quarantine queue (root cause fix)
Enqueue()on the quarantine queueCalendar icons
color-scheme: darkadded to CSS:rootso native date inputs render with visible iconsColor variables
--warningreferences fixed to--warn(the actual defined variable)--inforeferences fixed to#58a6ff(GitHub dark blue)Test plan
go test ./...all green