-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ActionList] Remove focus ring on click #3649 #4034
Conversation
馃憢 Thanks for opening your first pull request. A contributor should give feedback soon. If you haven鈥檛 already, please check out the contributing guidelines. |
a3ed8f1
to
313a2e1
Compare
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.
A couple odd things I think need to be left out of this PR
313a2e1
to
574a777
Compare
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.
574a777
to
9fad616
Compare
update unrelease.md
25f186e
to
0ba34aa
Compare
Co-authored-by: Kyle Durand <kyledurand@users.noreply.github.com>
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.
Thanks for doing this and following through @skrabakl-shopify!
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.
馃帀 This looks great, thank you @skrabakl-shopify !
Awesome! |
WHY are these changes introduced?
Fixes #3649
WHAT is this pull request doing?
Applied the same approach used on Polaris Button component.
How to 馃帺
馃枼 Local development instructions
馃棐 General tophatting guidelines
馃搫 Changelog guidelines
ActionList.focus-ring.demo.mov
馃帺 checklist
README.md
with documentation changes