Skip to content

fix: accessibility improvements#226

Merged
stdavis merged 2 commits intomainfrom
fix/diagon-a11y
Mar 23, 2026
Merged

fix: accessibility improvements#226
stdavis merged 2 commits intomainfrom
fix/diagon-a11y

Conversation

@stdavis
Copy link
Copy Markdown
Member

@stdavis stdavis commented Mar 20, 2026

@stdavis stdavis enabled auto-merge (rebase) March 20, 2026 22:26
@github-actions
Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit 39357a6):
Firebase Preview: https://ut-dts-agrc-broadband-dev--fix-diagon-a11y-5lsi9zmz.web.app
(expires "2026-04-03T22:27:37.121401163Z" )

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses accessibility-related improvements (issue #813) and unblocks local development by adding a Vite module alias workaround.

Changes:

  • Add a Vite resolve.alias entry to work around a dev-time dependency deep-import mismatch.
  • Update the app title heading in the header from h2 to h1.
  • Ignore a local Playwright MCP output directory.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
vite.config.ts Adds an alias to normalize a problematic deep import during Vite dev startup.
src/main.tsx Improves document heading semantics by using a top-level h1 for the app title.
.gitignore Adds an ignore rule for .playwright-mcp/.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@stdavis stdavis requested a review from jacobdadams March 20, 2026 23:13
Copy link
Copy Markdown
Member

@jacobdadams jacobdadams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Harry Potter Approved™

@stdavis stdavis merged commit b5adb2f into main Mar 23, 2026
10 checks passed
@stdavis stdavis deleted the fix/diagon-a11y branch March 23, 2026 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants