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 using css flexbox

Milanβ€’ 260

@milanjelic92

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


I have a little problem with overlay but i succeded as i put it in a empty div but i think it can be done in a different way. Feel free to check out my solution and give me some tips...

Community feedback

P
Mina Makhloufβ€’ 770

@Minamakhlouf

Posted

Hi Milan,

Your design is very good but there are a few things you can do to make it better.

  1. The card is too small for a Desktop. Something that you can do is increase the padding and the size of the text to make it bigger.

  2. The card isn't centered on the page like it is in the example. What you can do is make the <main> element a height of 100vh, give it a display of flex, and then justify-content and align-items center to make the card centered on the page.

Aside from these 2 minor changes, congratulations on the project.

Marked as helpful

0
Bizzareβ€’ 1,320

@godmayhem7

Posted

hey easy going ,nice one you've got thereπŸ‘ so here are some tips i think you should use to better your code as a programmer you should be considerate to how your client see his/her screen your website font was kinda tiny and every thing looked small the rest was okay πŸ‘ŒπŸ‘Œ hope this was helpful

Marked as helpful

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