-
Notifications
You must be signed in to change notification settings - Fork 49
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
Add new ButtonCard component #961
Conversation
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 good! Just a couple of minor nits below.
<div className="newspack-dashboard-card__contents"> | ||
<Icon icon={ plugins } /> | ||
<div className="newspack-dashboard-card__header"> | ||
<Card className="newspack-dashboard-card"> |
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.
Why a regular Card
here—is it just so we can use the strong
in the message? Wonder if we could adapt the ButtonCard
to render children
if it exists instead of (or in addition to) the desc
prop?
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.
It's not a link so it doesn't make sense to use a ButtonCard
. It's also temporary and the Google oauth connection will be moved elsewhere.
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.
Approving because my remaining question isn't a blocker if you don't think it's worth a refactor. Also need to resolve the conflicts before merging.
# [1.40.0-alpha.1](v1.39.0...v1.40.0-alpha.1) (2021-05-18) ### Bug Fixes * replace WP User Avatar with Simple Local Avatars ([#966](#966)) ([f980412](f980412)) * **oauth:** wpcom token saving ([24052d6](24052d6)) * **progress-bar:** radius when having headings ([#963](#963)) ([8347362](8347362)) * loading quiet anim time/height and margin when admin menu is folded ([#958](#958)) ([f297780](f297780)) ### Features * add an AutocompleteWithSuggestions component for reusability ([#952](#952)) ([460728d](460728d)) * add new ButtonCard component ([#961](#961)) ([eff9edf](eff9edf)) * add reusable blocks extended as supported plugin ([#968](#968)) ([10f9758](10f9758))
🎉 This PR is included in version 1.40.0-alpha.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
# [1.40.0](v1.39.0...v1.40.0) (2021-05-18) ### Bug Fixes * replace WP User Avatar with Simple Local Avatars ([#966](#966)) ([f980412](f980412)) * **oauth:** wpcom token saving ([24052d6](24052d6)) * **progress-bar:** radius when having headings ([#963](#963)) ([8347362](8347362)) * loading quiet anim time/height and margin when admin menu is folded ([#958](#958)) ([f297780](f297780)) ### Features * add an AutocompleteWithSuggestions component for reusability ([#952](#952)) ([460728d](460728d)) * add new ButtonCard component ([#961](#961)) ([eff9edf](eff9edf)) * add reusable blocks extended as supported plugin ([#968](#968)) ([10f9758](10f9758))
🎉 This PR is included in version 1.40.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
All Submissions:
Changes proposed in this Pull Request:
This PR adds a brand new component:
ButtonCard
to be used in the Dashboard and the Onboarding (for the upcoming migration step).How to test the changes in this Pull Request:
Other information: