web3modal icon indicating copy to clipboard operation
web3modal copied to clipboard

[feature] Add Alt-Text Tags to Resolve Critical WCAG Accessibility Issues in Web3Modal

Open yungzyad opened this issue 1 year ago • 4 comments

What problem does this new feature solve?

Users that depend on assistive technologies like screen readers are unable to use Web3Modal since icon buttons and other UI elements lack alt-text tags. This minor fix will reduce barriers to entry for millions of web3 users!

See here for an accessibility report for the Web3Modal demo on lab.web3modal.com/library/wagmi-all/

Describe the solution you'd like

Add descriptive aria-label tags to icon buttons and other UI elements in Web3Modal, or another WCAG approved solution.

References to some of the offending UI elements: Help Icon Button https://github.com/WalletConnect/web3modal/blob/main/packages/scaffold-ui/src/partials/w3m-header/index.ts#L208 Close Icon Button https://github.com/WalletConnect/web3modal/blob/main/packages/scaffold-ui/src/partials/w3m-header/index.ts#L165 Header Text https://github.com/WalletConnect/web3modal/blob/main/packages/scaffold-ui/src/partials/w3m-header/index.ts#L184

yungzyad avatar Aug 13 '24 19:08 yungzyad

Thank you for the valuable feedback!

rtomas avatar Aug 14 '24 01:08 rtomas

https://linear.app/walletconnect/issue/APKT-876/[feature]-add-alt-text-tags-to-resolve-critical-wcag-accessibility

rtomas avatar Aug 14 '24 01:08 rtomas

Hey @rtomas, I would like to work on the issue, can you please assign it to me if not resolved?

p1kalys avatar Sep 01 '24 06:09 p1kalys

Could you please fix these as well, thank you! If we run axe test we see them as critical.

  1. ARIA dialog and alertdialog nodes should have an accessible name Description: Ensures every ARIA dialog and alertdialog node has an accessible name Help: ARIA dialog and alertdialog nodes should have an accessible name Help URL: https://dequeuniversity.com/rules/axe/4.5/aria-dialog-name?application=axeAPI Impact: serious Tags: cat.aria, best-practice Element(s): Html:

    Selector(s): w3m-modal, #w3m-modal

    Solution - Add aria-label

  2. ARIA attributes must conform to valid values Description: Ensures all ARIA attributes have valid values Help: ARIA attributes must conform to valid values Help URL: https://dequeuniversity.com/rules/axe/4.5/aria-valid-attr-value?application=axeAPI Impact: critical Tags: cat.aria, wcag2a, wcag412 Element(s): Html:

    Selector(s): w3m-modal, #w3m-modal

    Solution - add aria-hidden value true/false when modal is opened and closed respectively.

VamshikrishDanam avatar Sep 03 '24 20:09 VamshikrishDanam