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

fix(Geosuggest): fix accessibility #489

Merged
merged 3 commits into from
Aug 3, 2023

Conversation

joaogarin
Copy link
Contributor

@joaogarin joaogarin commented Oct 18, 2021

Description

This PR fixes an accessibility with the current solution

  • listbox field does not have a proper label

Screenshot from 2021-10-18 12-41-19

Checklist

  • All tests passing
  • Created tests which fail without the change (if possible)
  • Extended the README / documentation, if necessary (N/A)
  • Commits and PR follow conventions

Fixes :

@joaogarin
Copy link
Contributor Author

Hello @yfr any chance we could give the accessibility PR's a quick look ? ;) Would be great thank you!

Copy link
Contributor

@plumdumpling plumdumpling left a comment

Choose a reason for hiding this comment

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

Nice one. Thank you!

@plumdumpling plumdumpling merged commit b5a32e7 into ubilabs:master Aug 3, 2023
@fcabreraupgrade
Copy link

Hey @plumdumpling! Thanks a lot for finally merge this fix!

Any chance you could release the accessibility fix with a patch version?

@plumdumpling
Copy link
Contributor

Hey @fcabreraupgrade! We would like to release the fix together with some more fixes and smaller updates. We're planning to have a new release ready this week.

@plumdumpling
Copy link
Contributor

@fcabreraupgrade we released a new version including this fix under our organization scope @ubilabs/react-geosuggest and will continue there from now on.

@fcabreraupgrade
Copy link

@plumdumpling thanks a lot!

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