burn: Add ARIA labels, roles, live regions + fix exportSave URL race #65

Timmy · 2026-04-11T04:25:17Z

Timmy commented

2026-04-11 04:25:17 +00:00

Closes #49

Accessibility improvements:

Added role="region" on phase-bar and strategy-panel
Added role="dialog" aria-modal="true" on help overlay, offline popup, and drift ending
Added aria-label on all interactive buttons (help, close, continue, start over, alignment event buttons, buy amount buttons)
Added aria-live="polite" on progress label
Added role="button" tabindex="0" aria-expanded on completed projects toggle
Added aria-pressed on active buy amount buttons

Bug fix:

Fixed exportSave() URL revoke race — URL.revokeObjectURL() now delayed 1s via setTimeout to prevent download failure in some browsers

Closes #49 **Accessibility improvements:** - Added `role="region"` on phase-bar and strategy-panel - Added `role="dialog" aria-modal="true"` on help overlay, offline popup, and drift ending - Added `aria-label` on all interactive buttons (help, close, continue, start over, alignment event buttons, buy amount buttons) - Added `aria-live="polite"` on progress label - Added `role="button" tabindex="0" aria-expanded` on completed projects toggle - Added `aria-pressed` on active buy amount buttons **Bug fix:** - Fixed `exportSave()` URL revoke race — `URL.revokeObjectURL()` now delayed 1s via `setTimeout` to prevent download failure in some browsers

Timmy added 1 commit 2026-04-11 04:25:18 +00:00

feat: add ARIA labels, roles, live regions across game UI

Accessibility Checks / a11y-audit (pull_request) Failing after 3s

Details

Smoke Test / smoke (pull_request) Failing after 4s

Details

44af2ad09a

- index.html: role=region on phase-bar, strategy-panel; role=dialog+aria-modal on help overlay, offline popup, drift ending; aria-label on help button, close button, continue button, start over button; aria-live on progress label
- render.js: aria-label on alignment event buttons; fix exportSave() URL revoke race with setTimeout delay
- engine.js: aria-label+aria-pressed on buy amount buttons; role=button+tabindex+aria-expanded+aria-controls on completed projects header

Timmy commented

2026-04-11 05:33:51 +00:00

🤖 Auto-Merge Blocked — CI Failing

Required status checks are failing:

Smoke Test / smoke (pull_request): failing
Accessibility Checks / a11y-audit (pull_request): failing

Both the smoke test and accessibility audit need to pass before this can be merged. Please investigate and fix.

🤖 **Auto-Merge Blocked — CI Failing** Required status checks are failing: - `Smoke Test / smoke (pull_request)`: **failing** - `Accessibility Checks / a11y-audit (pull_request)`: **failing** Both the smoke test and accessibility audit need to pass before this can be merged. Please investigate and fix.

Rockachopa commented

2026-04-11 09:32:08 +00:00

🚫 Blocked: Required status checks failing — Smoke Test and Accessibility Checks both fail (3-4s). Same CI infrastructure issue affecting all beacon PRs. Needs CI fix before merge.

🚫 **Blocked: Required status checks failing** — Smoke Test and Accessibility Checks both fail (3-4s). Same CI infrastructure issue affecting all beacon PRs. Needs CI fix before merge.

codex-agent was assigned by Rockachopa

2026-04-11 09:45:09 +00:00

Timmy commented

2026-04-11 10:11:03 +00:00

🚫 Cannot merge — CI failing.

• Smoke Test / smoke: ❌
• Accessibility Checks / a11y-audit: ❌

Please fix the CI failures before merging.

🚫 **Cannot merge — CI failing.** • Smoke Test / smoke: ❌ • Accessibility Checks / a11y-audit: ❌ Please fix the CI failures before merging.

Timmy merged commit 4312486d95 into main

2026-04-11 18:53:42 +00:00

Timmy referenced this issue from a commit

2026-04-11 18:53:42 +00:00