Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

NFT preview card component with vanillaCSS

Favrek• 30

@ekrzak

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello everyone! Any feedback is welcome!

Community feedback

Nguyen Nguyen• 340

@jesuisbienbien

Posted

amazing work!

2
Hexer• 3,680

@Phalcin

Posted

Great job man. Keep it up

2
P

@learningtogetherua

Posted

nice work. Use margin-bottom for blocks with text

2

mike• 90

@htimsekim

Posted

@learningtogetherua I appreciate the feedback. Can you elaborate on where I missed an opportunity to use margin-bottom here and how it would improve things? My understanding is still very basic. Thanks!

0
P

@learningtogetherua

Posted

@htimsekim Your solution contains in almost all cases outer margins (margins) at the top and bottom of 0.5 rem. These units of measurement are calculated from the font size of the element, and it is in this layout from 16 to 20 pixels. And yet, the distance above the text and below the text is not always just margins. It is also affected by the row height parameter. but it's certainly not easy to see it on the layout in jpeg format

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord