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
adds card first design-system rules #602
base: release/1.0.0
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## release/1.0.0 #602 +/- ##
===============================================
Coverage 100.00% 100.00%
===============================================
Files 85 85
Lines 1546 1546
Branches 538 538
===============================================
Hits 1546 1546 ☔ View full report in Codecov by Sentry. |
fe0073b
to
a024c90
Compare
a024c90
to
415b229
Compare
example/src/components/CardDemo.tsx
Outdated
@@ -17,6 +17,7 @@ import { faAngleRight } from '@fortawesome/free-solid-svg-icons'; | |||
import './cardDemo.scss'; | |||
|
|||
export const CardDemo = () => { | |||
console.log('HELLO'); |
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.
please remove this log
|
||
.dcx-card-image { | ||
max-width: 100%; | ||
max-height: 100%; |
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.
do we want to use any padding here?
padding: .....
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.
Now we have this gap => column-gap: token('spacing-y-card_box');
d1dad72
to
4ec2813
Compare
4ec2813
to
01783fd
Compare
No description provided.