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

[gatsby-theme-docz]: add aria-label attribute to theme button #1383

Merged
merged 1 commit into from
Feb 9, 2020

Conversation

alexandre-lelain
Copy link
Contributor

@alexandre-lelain alexandre-lelain commented Feb 9, 2020

issue: #1359

Description

This PR adds an aria-label attribute on the Header's button responsible for switching between light and dark mode for screen readers.

Review

  • Check that there is an aria-label="Switch to light|dark mode" attribute on the theme button in the Header.

@rakannimer
Copy link
Contributor

This is cool, thanks @alexandre-lelain !

@rakannimer rakannimer merged commit e81286a into doczjs:master Feb 9, 2020
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.

2 participants