Skip to content
Submitted over 1 year ago

NFT preview card component

Beshoy Maher•150
@BeshoyMaherr
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...NFT preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Tell me how can i improve my self

Community feedback

  • IryDev•1,580
    @IryDevPosted over 1 year ago

    Hey, well done for completed this challenge😁

    I have some suggestions to help you to improve your solution :

    • Put the main content of your website into the <main>tag in order to improve accessibility
    • Another alternative to center in your card :
    • Give the body a min-height of 100vh
    • Add the property , display: flex; justify-content: center; align-items: center; to make the card center on the vertical axis and horizontal axis

    HTML :

    <body>
      <main>
            The main content of your website
      </main>
</body>

    CSS :

    body {
  min-height: 100vh;
  display: flex;
  align-items: center;
  justify-content: center;
}

    I hope you'll find this helpful, and your solution is really good 😄

    Marked as helpful
    0
    Beshoy Maher•150
    @BeshoyMaherrPosted over 1 year ago

    @IryCode Thanks a lot, I appreciate that

    0
    IryDev•1,580
    @IryDevPosted over 1 year ago

    @BeshoyMaherr You're welcome 😄

    0
  • Morin Sultan•310
    @SoulOfMoPosted over 1 year ago

    Nice work man. You can set the height of the body to 100vh to stop it from overflowing. Also you can use the main tag instead of <div class="parent">. Your site is responsive which is very good.

    0
    Beshoy Maher•150
    @BeshoyMaherrPosted over 1 year ago

    Thanks @SoulOfMo

    0

Please log in to post a comment

Log in with GitHub

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