Skip to content

Button Component: update to use non-deprecated attributes #23242

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.

Sign up for GitHub

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

Merged
oskosk merged 11 commits into from
Mar 7, 2022
Merged

Button Component: update to use non-deprecated attributes #23242

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
5f484f8
Blocks: update secondary buttons to use non-deprecated attribute.
jeherve Mar 4, 2022
7d8ede2
Remove isLarge, it is not used anymore.
jeherve Mar 4, 2022
32d4faa
Update changelog entry
jeherve Mar 4, 2022
1346801
Replace all isPrimary references
jeherve Mar 4, 2022
e26a5f0
Update the logic to be simpler
jeherve Mar 4, 2022
ed89c6b
Move from isSecondary to variant
jeherve Mar 4, 2022
4fa2f08
Move from isLink to variant
jeherve Mar 4, 2022
63ce6f5
Move from isTertiary to variant
jeherve Mar 4, 2022
068d579
Update usage of the Button component in our other projects
jeherve Mar 4, 2022
863ed5c
Bump versions
jeherve Mar 4, 2022
bb8fe46
Merge branch 'master' into update/buttons-deprecated-isdefault
oskosk Mar 7, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Merge branch 'master' into update/buttons-deprecated-isdefault
  • Loading branch information
@oskosk
oskosk committed Mar 7, 2022
commit bb8fe46d4ebfe433b249ea5667c4445e7029fe2c
1 change: 1 addition & 0 deletions projects/packages/my-jetpack/_inc/components/product-detail-button/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ const ProductDetailButton = ( { children, className, href, isLoading, onClick, i
className={ className }
href={ href }
variant={ isPrimary ? 'primary' : 'secondary' }
disabled={ isLoading }
>
{ isLoading ? <Spinner /> : children }
</Button>
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.