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

Show foucs ring on checkbox #2242

Closed
wants to merge 2 commits into from
Closed

Conversation

flip-it
Copy link
Contributor

@flip-it flip-it commented Feb 25, 2022

When navigating between inputs with keyboard, there is no focus ring on checkboxes. This can be solved by replacing opacity: 0 with appearance: 'none' on the native element. One caveat is that IE 11 doesn't support this property. But maybe it's better to favour accessibility than an outdated browser?

@necolas
Copy link
Owner

necolas commented Mar 1, 2022

The same change is needed for Switch too

@flip-it
Copy link
Contributor Author

flip-it commented Mar 1, 2022

Indeed. I added it.

@necolas necolas added this to the 0.18: StyleSheet milestone Mar 1, 2022
necolas pushed a commit that referenced this pull request Mar 2, 2022
@necolas necolas added the has: pr Subject of a pull request label Mar 2, 2022
@necolas necolas mentioned this pull request Mar 23, 2022
@necolas necolas closed this in e7a99f5 Jun 10, 2022
rnike pushed a commit to VeryBuy/react-native-web that referenced this pull request Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has: pr Subject of a pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants