Skip to content

Add aria-label to language selector button#2725

Merged
squidfunk merged 1 commit intosquidfunk:masterfrom
Stanzilla:patch-7
Jun 6, 2021
Merged

Add aria-label to language selector button#2725
squidfunk merged 1 commit intosquidfunk:masterfrom
Stanzilla:patch-7

Conversation

@Stanzilla
Copy link
Copy Markdown
Contributor

@Stanzilla Stanzilla commented May 31, 2021

Adds aria-label to language selector buttons since otherwise screen readers are not able to read it.

@Stanzilla Stanzilla force-pushed the patch-7 branch 2 times, most recently from 7f677de to 569ed67 Compare May 31, 2021 16:34
@squidfunk
Copy link
Copy Markdown
Owner

Thanks for the PR. I’d say we call it „Select language“ and add i18n support via language.select.

@squidfunk squidfunk merged commit 2a5ba14 into squidfunk:master Jun 6, 2021
@squidfunk
Copy link
Copy Markdown
Owner

squidfunk commented Jun 6, 2021

I've done a little cleanup in 11fcf4f:

  • Changed my mind on the translation key again
  • Added an aria-label to the version selector

@Stanzilla Stanzilla deleted the patch-7 branch June 6, 2021 14:33
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