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

Responsive NFT preview card component Using CSS Flex Box

Chamsโ€ข 160

@Chams-sat

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

Agnik Bakshiโ€ข 480

@Agnik7

Posted

Hi,

Congratulations on completing this challenge!!๐ŸŽ‰๐ŸŽ‰ While going through your code, I found certain things that might be of help to you.

  • For the styling of the body tag, you can actually omit position: relative and width:100%. This is because even if you don't define the width of any container element, it's width is taken 100% by default. As for the position:relative, it's just unnecessary. Instead of defining the height as 100vh, define the min-height as 100vh for better responsiveness. min-height:100vh.

  • For the .card, instead of assigning the width, assign the max-width. This way, you set a limit to the width, and at the same time, not keep the width constant. Do not define the height of the card. This way, you can ensure that the card takes up as much space as it requires, and no extra space is wasted. Additionally, you can also omit the media query, as it will not be required once you remove the declaration of height.

  • In order to prevent overflow, set only the top margin of the info section as 1rem.

.info{
margin-top:1rem;
}

Hope this feedback helps you. Have a nice day!!!

Marked as helpful

0

Chamsโ€ข 160

@Chams-sat

Posted

@Agnik7 Thank you ๐Ÿ˜Š

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