Skip to content
This repository has been archived by the owner on Oct 20, 2022. It is now read-only.

Make tooltip accessable #474

Open
Myoxocephalus opened this issue Dec 2, 2018 · 1 comment
Open

Make tooltip accessable #474

Myoxocephalus opened this issue Dec 2, 2018 · 1 comment

Comments

@Myoxocephalus
Copy link
Contributor

Hi!

Issue

  1. The tooltip component is currently hard to access with a keyboard since it´s triggered by onMouseEnter.
  2. It´s hard to find and read the text for a screen reader.

Sugested soultion

  1. Add onFocus as trigger.
  2. Add role="tooltip" to the tooltip element.

If you think the suggested solutions is fine i can send a PR.

@bstream
Copy link
Contributor

bstream commented Dec 4, 2018

Hey!

I think your suggestions sound good :) The only thing we need to keep in mind is how/if it will affect the current functionality!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants