-
Notifications
You must be signed in to change notification settings - Fork 27.9k
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
Add border to status bar items on keyboard focus #142665
Conversation
Overall, this is looking good and like that we are finally using a proper focus border in the status bar!
@bpasero I wonder if there's anything we can do here to just add an additional 1px padding to increase the status bar for themes that use this? things already shift when you switch to a theme with borders so I don't see this being a big issue |
Just tried this out now and this is great work @daviddossett 👏 |
This PR fixes #141709 by adding an outline to status bar items when navigating through them with the keyboard.
Additionally, the focus outline offset changes slightly for themes that include a
statusBar.border
color.153275287-06205734-e569-4b9b-b3a1-2beaa4169638.mp4