-
Notifications
You must be signed in to change notification settings - Fork 124
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
Shift renders focus ring #148
Comments
Hey @michax, we currently turn on focus-visible anytime a user presses any keyboard key. Our thinking is that this is a signal that the user is in "keyboard mode." Can you tell me a bit more about what you were building that caused you to see this as an error? We're always interested in use cases to help us shape the design. |
Just chiming in, but shift (or any keyboard event) also causes Chrome's default focus ring to appear, as well. Personally, it seems somewhat aggressive. Consider the case where a user is mouse-navigating and then starts entering a keyboard shortcut, e.g. MOD-A to select all text. The focused element becomes highlighted at the start, the instant MOD is first pressed. For someone who hasn't been using the keyboard, it seems incongruous. |
I agree that it may be a bit aggressive. @alice and I were chatting about it this past week. It's unclear (to me at least) if the current behavior was a conscious decision by the Chrome team, or if it's merely a side effect of how focus ring drawing is implemented. Neither Firefox nor Safari has this same behavior, but if I recall correctly, they also don't technically focus the button when you mouse click on it, whereas Chrome does. For now, what's in the polyfill matches the |
During testing I've noticed that that shift press alone activates focus ring. It doesn't seem like correct behaviour. I would expect it to only appear after tab or tab + shift.
The text was updated successfully, but these errors were encountered: