-
Notifications
You must be signed in to change notification settings - Fork 190
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
NO-JIRA: Added submit to loading button props #3574
Conversation
Visit https://backpack.github.io/storybook-prs/3574 to see this build running in a browser. |
Visit https://backpack.github.io/storybook-prs/3574 to see this build running in a browser. |
Browser supportIf this is a visual change, make sure you've tested it in multiple browsers. |
Visit https://backpack.github.io/storybook-prs/3574 to see this build running in a browser. |
</BpkButtonV2> | ||
); | ||
|
||
const SubmitLoadingButtonExample = (props: any) => ( |
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.
ohh this one should probably sit in the bpk-component-loading-button example file 😄
Visit https://backpack.github.io/storybook-prs/3574 to see this build running in a browser. |
Visit https://backpack.github.io/storybook-prs/3574 to see this build running in a browser. |
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.
Awesome! I think it'd be a patch
rather than minor
though? Wdyt? 🤔
Yeah I agree, thanks! |
Minor change to add the submit prop to the loading button props. Due to confusion in the past, it's probably best to make the submit prop type for the loading button explicit for consumers of Backpack as in the past, some users have set the type to be submit without knowing the submit prop exists and inherits the props from BpkButton via ...rest
Remember to include the following changes:
[KOA-123][BpkButton] Updating the colour
README.md
(If you have created a new component)README.md