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 default card visualizations to SetupBanner and additional clean-up and styling. #6928
Add default card visualizations to SetupBanner and additional clean-up and styling. #6928
Conversation
beginnings of hooking up card to cards
making default props optional add example
Added additional style locations Added final styling Added animations Adding strokeLineCap to IRawStyleBase
show all the different card states
remove button and link styles correct setup card skew
…c-react into brandonthomas/setupCards
* SVG: The stroke-linecap attribute defines the shape to be used at the end of open subpaths when they are stroked. | ||
* See SVG 1.1 https://www.w3.org/TR/SVG/painting.html#LineCaps | ||
*/ | ||
strokeLinecap?: ICSSRule | 'butt' | 'round' | 'square'; |
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.
cool
Codecov Report
@@ Coverage Diff @@
## master #6928 +/- ##
=======================================
Coverage 74.89% 74.89%
=======================================
Files 859 859
Lines 24451 24451
Branches 5383 5383
=======================================
Hits 18312 18312
Misses 5675 5675
Partials 464 464 Continue to review full report at Codecov.
|
packages/dashboard/src/components/SetupBanner/SetupCards.types.ts
Outdated
Show resolved
Hide resolved
packages/dashboard/src/components/SetupBanner/SetupCard.styles.ts
Outdated
Show resolved
Hide resolved
packages/dashboard/src/components/SetupBanner/SetupCard.styles.ts
Outdated
Show resolved
Hide resolved
Referencing font size styles
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.
@dzearing , once the checks finish, who can help me merge this in? |
🎉 Handy Links: |
🎉 Handy Links: |
Pull request checklist
$ npm run change
working with DZearing to get a change file as my environment i barfing on this. If we can't figure it out he'll submit one for me.
Description of changes
Focus areas to test
Microsoft Reviewers: Open in CodeFlow