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

Mobile first - flexbox

P
Dzik0 180

@Dzik0

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@DebabrataBanik

Posted

Your design looks amazing! Few things you might want to look at:

  1. Remove any default margin and padding from the body and set min-height: 100vh. This ensures the layout spans the full viewport height and eliminates unnecessary scrolling.
  2. On your img_container you can add display: flex to make sure the image inside is centered and covers the full containers space.

Marked as helpful

1

P
Dzik0 180

@Dzik0

Posted

@DebabrataBanik Thank you so much for your feedback, gonna defenietly start using the min-height and gonna remove any default margin/padding!

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