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

Tooltip is not provided for 'next/ previous' icons #274

Open
robertmausser opened this issue Aug 24, 2023 · 1 comment
Open

Tooltip is not provided for 'next/ previous' icons #274

robertmausser opened this issue Aug 24, 2023 · 1 comment

Comments

@robertmausser
Copy link

robertmausser commented Aug 24, 2023

Subject of the issue/enhancement/features

Tooltip is not provided for 'next/ previous' icons.

Your environment

  • 5.31.10
  • all browsers
  • all OSes

Steps to reproduce

Use screen reader to move to previous/next buttons.

Expected behaviour

Tooltip should be provided for previous/next icons.

Actual behaviour

Tooltip is not provided for previous/next icons.

Screenshots (if you can)

https://app.screencast.com/JaMrV2j8R3oIB

How to Fix

"1. Create a

element, give it a role=""tooltip"" and an id=""IDREF"". Then, give it a brief description of the action.
2. This element has to be accessible by focusing as well as hovering over. Both functionalities can be provided using CSS.
3. The tooltip has to be visually hidden using CSS.
4. Associate the tooltip with its button using aria-labelledby.
5. After applying this keyboard-accessible tooltip, consider removing the title attribute from the element.

Refer to:
https://inclusive-components.design/tooltips-toggletips/
https://a11yproject.com/posts/how-to-hide-content/"

@kirsty-hames
Copy link
Contributor

For consistency with other Adapt buttons we could use the core tooltip API for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants