-
Notifications
You must be signed in to change notification settings - Fork 55
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
fix(controls): improve focus style to meet WCAG 2.2 AA #267
Conversation
✅ Deploy Preview for stacks-editor ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@dancormier What do you think of adding
This reference will then be moved to the devDependencies object the same way we do for I am suggesting this because the release containing the changes on this PR (probably |
I took the liberty to cut a stacks classic release candidate and also add 2.1.0 as peer deps in this PR. |
…sometimes there a small breaking changes in minor releases)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Work as advertised. Great work @dancormier
I am approving but let's remember we need to change stacks classic dep to 2.1.0 before merging and then cut the release.
Closes STACKS-542
This PR updates focus styles for buttons that are editor controls.