bs-ux-10: contextual empty states in Mission Control dashboard#176
Merged
dgarson merged 2 commits intofeat/horizon-ui-completefrom Feb 24, 2026
Merged
bs-ux-10: contextual empty states in Mission Control dashboard#176dgarson merged 2 commits intofeat/horizon-ui-completefrom
dgarson merged 2 commits intofeat/horizon-ui-completefrom
Conversation
Audited and remediated WCAG 2.1 AA violations across MissionControlDashboard and AgentTopologyMap (FindingDetailModal and DiscoveryRunHistory do not exist on this branch — documented in audit report). MissionControlDashboard.tsx (18 violations fixed): - Skip link + <main id> landmark added - All decorative icons: aria-hidden="true" throughout - SeverityIcon variants: role="img" + aria-label per severity level - ToolCallsPanel status icons: CheckCircle/XCircle labeled for AT - SessionStatusBadge: role="status"; dot aria-hidden - AlertFeed filters: aria-pressed + focus-visible:ring-2 + role="group" - Approve/Deny buttons: aria-label with action context + focus ring - Alert feed: role="log" + aria-live="polite" - LiveStatusBar: aria-live="polite" region - Panels converted to <section aria-label> landmarks AgentTopologyMap.tsx (14 violations fixed): - Skip link + <main id> landmark added - SVG: role="application" + aria-label for keyboard context - NodeShape: tabIndex=0, role="button", aria-label, aria-pressed, onKeyDown - focusedId state + visible focus ring (glow) distinct from selected state - DetailPanel: <aside role="complementary" aria-label> - DetailPanel close button: aria-label="Close details panel" - Escape key handler: closes detail panel (no keyboard trap) - Token load bar: proper role="progressbar" with ARIA values - Live/Paused toggle: aria-pressed + descriptive aria-label - All toolbar/legend icons: aria-hidden="true" - SVG edges/grid/dots: aria-hidden="true" AgentOutputDiffViewer.tsx (pre-existing build fix, not WCAG): - Removed 9 non-existent lucide-react imports blocking build - Renamed Map import to MapIcon to stop shadowing JS built-in Map docs/WCAG_AUDIT_REPORT.md: full audit report with violation inventory, fixes applied, remaining items, and WCAG 2.1 AA coverage matrix
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
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
Validation
Work Item