Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve accessibility with ARIA attributes and semantic HTML #792

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sheldon-im
Copy link

Accessibility Improvements

This pull request enhances the application's accessibility by implementing ARIA attributes and semantic HTML throughout the codebase.

Changes

  • Added appropriate ARIA attributes (aria-label, aria-live, aria-hidden, etc.) to improve screen reader compatibility
  • Implemented semantic HTML elements with proper roles
  • Enhanced keyboard navigation support
  • Improved focus management for interactive elements
  • Added descriptive labels for UI components

Summary of Changes

  • 49 files changed
  • 921 insertions
  • 495 deletions

Major changes were made to:

  • AlertComp.svelte (503 changes)
  • ChatList.svelte (147 changes)
  • MobileFooter.svelte (103 changes)
  • GridCatalog.svelte (90 changes)
  • ChatScreen.svelte (49 changes)
  • App.svelte (45 changes)
  • MobileHeader.svelte (42 changes)

These improvements make the application more accessible to users with disabilities and comply better with WCAG (Web Content Accessibility Guidelines) standards.

@kwaroran
Copy link
Owner

kwaroran commented Mar 17, 2025

Some aria messages are written only in Korean, which can confuse non-Korean users. I suggest using writing in english or using language variable from the src/lang/index.ts

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.

2 participants