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/switch aria labelledby #219

Closed
wants to merge 2 commits into from
Closed

Conversation

juhomakkonen
Copy link
Collaborator

Fix to aria-labelledby values

Replace previous aria-labelledby values, that were long strings with shorter cleaner strings.


Breakdown:

Kategoria-otsikko

  1. src/views/MobilitySettingsView/components/MobilityToggleButton/MobilityToggleButton.js

    • Add new prop of type to be used in aria-labelledby
  2. src/views/MobilitySettingsView/MobilitySettingsView.js

    • Use the new prop.

@juhomakkonen juhomakkonen added the accessibility Improvements to accessibility label Jan 31, 2024
@juhomakkonen juhomakkonen requested a review from SanttuA January 31, 2024 13:58
@juhomakkonen juhomakkonen deleted the fix/switch-aria-labelledby branch April 24, 2024 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Improvements to accessibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant