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

More accessible links in the search suggestions #650

Closed
zner0L opened this issue Aug 18, 2021 · 3 comments · Fixed by #776
Closed

More accessible links in the search suggestions #650

zner0L opened this issue Aug 18, 2021 · 3 comments · Fixed by #776
Assignees
Labels
a11y hacktoberfest help wanted Extra attention is needed

Comments

@zner0L
Copy link
Member

zner0L commented Aug 18, 2021

Instead of making the entire container a link which will be read out in full by screen readers, only use the heading. Using a clever overlay, clicking the whole container can still be preserved for visual users, like suggested in this video: https://youtu.be/qKRq-tFixWE?t=2227

This has also been discussed in #590.

@zner0L zner0L added help wanted Extra attention is needed a11y hacktoberfest labels Aug 18, 2021
@Vidushi-Gupta
Copy link

Hi, could you please let me know on which page can I change the linked containers to linked headings? I'll be happy to contribute to this. Could you please assign it to me?

@zner0L
Copy link
Member Author

zner0L commented Oct 1, 2021

You can work on this if you want. We don't just want the headings to be links, we also want the the entire container to be clickable while also being semantically correct for the sake of accessibility. The youtube video I linked shows a fairly elegant solution I'd like to be implemented.

@Vidushi-Gupta
Copy link

Sure, I'll work on this. You can go ahead an assign this to me, if it requires me to be an assignee. Thank you! I look forward to contributing :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y hacktoberfest help wanted Extra attention is needed
Development

Successfully merging a pull request may close this issue.

2 participants