Skip to content

Fix a11y issues#159

Open
T-kesh wants to merge 4 commits into
Sorokit:mainfrom
T-kesh:fix-a11y-issues
Open

Fix a11y issues#159
T-kesh wants to merge 4 commits into
Sorokit:mainfrom
T-kesh:fix-a11y-issues

Conversation

@T-kesh

@T-kesh T-kesh commented Jun 28, 2026

Copy link
Copy Markdown
  • Updated AddressDisplay copy button to be focus-visible and always partially visible (opacity-50) for keyboard & mobile users, with proper ARIA label.
  • Implemented SorobanPanel with an accessible textarea, linking label via htmlFor/id and adding state handling.
  • Refactored ErrorBoundary to expose raw error messages only in development; production now shows a generic “See the browser console for details.” message, preserving UX and security.

All changes pass npm run build and npm run lint. Accessibility criteria WCAG 1.3.1, 1.4.3, and 2.4.7 are addressed.
closes #27

@drips-wave

drips-wave Bot commented Jun 28, 2026

Copy link
Copy Markdown

@T-kesh Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

fix(a11y): AddressDisplay copy button invisible to keyboard/mobile, SorobanPanel textarea missing label, ErrorBoundary leaks error.message in production

1 participant