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

1s-preview-card

@Mostafa-ninoo

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


all feedback is welcome

Community feedback

Ali Ahmed 680

@Dany-GitHub

Posted

it's better to remove the height from the main and center all contents in the body, by using flex or gird, like this

body {
   display: flex;
    justify-content: center;
    align-items: center;
}

I see you used box-sizing in body to reset its properties, you can choose all elements to reset their properties as well like this

* {
    box-sizing: border-box;
    padding: 0;
    margin: 0;
}

the * means assign all those properties to all elements Well done ❤✌, Happy coding

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