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

Select focus state not visible enough #659

Closed
Aniort opened this issue Apr 27, 2021 · 3 comments · Fixed by #1437
Closed

Select focus state not visible enough #659

Aniort opened this issue Apr 27, 2021 · 3 comments · Fixed by #1437
Assignees
Projects

Comments

@Aniort
Copy link
Contributor

Aniort commented Apr 27, 2021

hello,
threre's an issue when the focus enter a hml form, the focus status indicator is just displayed by a light gray to black 2px border. Even if it's enough for other f'orm controls (because the flashing cursor appears) it's not the case for a select. We should increase the focus visiblity by, for example, changing the backgroàund ceolor of' the select or anything else...

@yannickcornaille
Copy link
Member

I applied this kind of indicator to an Orange project with a "Brand Orange" focus like this :

Capture d’écran 2021-09-21 à 15 59 11

Maybe we could do this just for <select> or even all form components.

@Aniort
Copy link
Contributor Author

Aniort commented Oct 7, 2021

it's nice and visible enough (and it'll be ok with the next version of WCAG 2.2 ready by end of 2020)
we can do it for all the form elements!

@ffoodd
Copy link
Contributor

ffoodd commented Dec 10, 2021

FWIW :focus-visible does not apply to form elements, so using such strong focus indicator will make it visible both for mouse and keyboard users. Which is usually good, but still: that's why it's not the case for now.

@julien-deramond julien-deramond added this to Triage in v5.2.2 via automation Nov 7, 2022
@julien-deramond julien-deramond moved this from Triage to In progress in v5.2.2 Nov 7, 2022
@julien-deramond julien-deramond removed this from In progress in v5.2.2 Nov 23, 2022
@julien-deramond julien-deramond added this to Triage in v5.2.3 via automation Nov 23, 2022
@julien-deramond julien-deramond moved this from Triage to In progress in v5.2.3 Nov 23, 2022
v5.2.3 automation moved this from In progress to Done Dec 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

4 participants