Skip to content

Adding search input field labels using aria-label#3046

Merged
oprypin merged 2 commits intomkdocs:masterfrom
dmundra:search-labels
Apr 28, 2023
Merged

Adding search input field labels using aria-label#3046
oprypin merged 2 commits intomkdocs:masterfrom
dmundra:search-labels

Conversation

@dmundra
Copy link
Copy Markdown
Contributor

@dmundra dmundra commented Nov 18, 2022

Fixes #3045

@ultrabug
Copy link
Copy Markdown
Member

We actually sync the theme from its readthedocs upstream so maybe this should be added there first hand?

@dmundra
Copy link
Copy Markdown
Contributor Author

dmundra commented Nov 21, 2022

Ya I submitted a fix upstream as well readthedocs/sphinx_rtd_theme#1380

@dmundra
Copy link
Copy Markdown
Contributor Author

dmundra commented Nov 21, 2022

FYI, the upstream patch has been merged. Should this wait for a new release of sphinx_rtd_theme?

@oprypin

This comment was marked as abuse.

@Sparticuz
Copy link
Copy Markdown
Contributor

fyi: i've put a pr in for rtd 1.1.1 (#3058), though @dmundra's patch upstream will probably be on 1.1.2.

@dmundra dmundra deleted the search-labels branch April 28, 2023 14:30
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.

Missing labels for the search boxes in the readthedocs theme (accessibility issue)

4 participants