You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Add a new tone property to the Button component. The tone can be one of the following: default, critical, info, success or warning.
Full description
Currently, I can't have a Button component with the critical or info state if it is also the secondary or tertiary variant. To fix that let's add new prop tone to Button component so new API will look like this:
typeButtonProps={tone?: 'default'|'critical'|'info'|'success'|'warning'variant: 'primary'|'secondary'|'tertiary'}<Buttonvariant="primary"/>// if not provided tone is `default`<Buttonvariant="secondary"/><Buttonvariant="tertiary"/><Buttonvariant="primary"tone="critical"/><Buttonvariant="secondary"tone="critical"/><Buttonvariant="tertiary"tone="critical"/><Buttonvariant="primary"tone="info"/><Buttonvariant="secondary"tone="info"/><Buttonvariant="tertiary"tone="info"/><Buttonvariant="primary"tone="success"/><Buttonvariant="secondary"tone="success"/><Buttonvariant="tertiary"tone="success"/><Buttonvariant="primary"tone="warning"/><Buttonvariant="secondary"tone="warning"/><Buttonvariant="tertiary"tone="warning"/>
Adding those roles will require additional tokens to be created inside contract.
krzysztofzuraw
changed the title
[RFC]: Add new prop to Button that describes its role
[RFC]: Add new prop to Button that describes its tone
Dec 6, 2023
Short description of the feature
Add a new
tone
property to theButton
component. Thetone
can be one of the following:default
,critical
,info
,success
orwarning
.Full description
Currently, I can't have a
Button
component with thecritical
orinfo
state if it is also thesecondary
ortertiary
variant. To fix that let's add new proptone
toButton
component so new API will look like this:Adding those roles will require additional tokens to be created inside contract.
Acceptance Criteria
default
,critical
,warning
,info
andsuccess
Additional notes
No response
The text was updated successfully, but these errors were encountered: