-
Notifications
You must be signed in to change notification settings - Fork 15
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
Link to patch detail #2741
Link to patch detail #2741
Conversation
63373db
to
a3c3e96
Compare
a3c3e96
to
289274a
Compare
56568d8
to
121f28e
Compare
Please provide PR description |
@balanza I totally overlooked that, thank you |
(buttonSize) => { | ||
const content = faker.vehicle.vehicle(); | ||
render( | ||
<Button size={buttonSize} disabled> |
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.
Should the button be disabled
here?
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.
LOL nope my bad, thanks
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.
Looks great. Thanks for improving the button story as well!
This PR adds:
link
used to make standard effortless programmatic navigation without having to appease a11y guidelines every time