RajanCP• 80
@RajanCP
Submitted
cyber-m
@cyber-m
All comments
- cyber-m• 80
@cyber-m
Posted
@RajanCP Great job! I've noticed one little thing, that the style of the ordered list numbers doesn't match the design(for example look at the number 1 in the design you'll see it looks different)
0 - Md Zafer Eqbal• 50
@MdZaferEqbal
Submitted
cyber-m• 80@cyber-m
Posted
Great work! @MdZaferEqbal, Just one small thing padding should be larger on desktop
Marked as helpful
0 - lazydroide• 110
@lazydroide
Submitted
What challenges did you encounter, and how did you overcome them?
Making the font-size and the width of the card responsive without using media queries was the biggest challenge but the clamp function worked well.
cyber-m• 80@cyber-m
Posted
Great Job! Regarding the font size I think you can make it work also just using single clamp at the root and use rem units for all the other text elements.
Marked as helpful
0 - Brandon J. Wells• 30
@dev-bjwells
Submitted
cyber-m• 80@cyber-m
Posted
Hey Brandon! Great job!
I noticed just couple of small details in the p tag styling. According to the Figma design file, it should include:
letter-spacing: 0.2px; /* or with any alternative CSS unit e.g., rems (rem) */ line-height: 140%; /* should render as 21px it's slight smaller in your implementation */0
