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

Propose accessibility fixes for enhanceSelectElement #769

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

Conversation

oscarduignan
Copy link

@oscarduignan oscarduignan commented Dec 16, 2024

These proposed changes would fix the following two issues

#589
#752
#592

these are something that hmrc services get asked to fix in accessibility audits so the services have to patch their usages with some js, we've been investigating how to roll the fixes into hmrc-frontend to save services having to add it themselves, but it seems like the fixes would be quite simple to make here so thought we'd propose them upstream just in case it's useful

(apologies I didn't realise that a deploy preview could be done for external contributions so I didn't regenerate the dist version so the deploy preview doesn't include the fix so you can't actually try it out there)

Copy link

netlify bot commented Dec 16, 2024

Deploy Preview for accessible-autocomplete ready!

Name Link
🔨 Latest commit 310e0b5
🔍 Latest deploy log https://app.netlify.com/sites/accessible-autocomplete/deploys/676019af03db350008778eab
😎 Deploy Preview https://deploy-preview-769--accessible-autocomplete.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

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.

1 participant