NFT preview card component

Community feedback

• 

  P

  solvman•1,650

  @solvmanPosted 10 months ago

  Congratulation! Your NFT card looks great! I would suggest you use semantic HTML more. For example, your first div

  <div class="main-container">...</div>
  

  could be replaced with a semantic version of

  <main>...</main>
  

  To place your card in the middle of the screen you could use:

  * {
      margin: 0;
  }
  
  html,
  body { 
      height: 100vh;
  }
  
  body {
      display: grid;
      place-content: center;
  }
  

  If you use display: grid, content will stretch automatically, so you must set max-width and/or min-width or the exact container size. Otherwise, you could use Flexbox:

  * {
      margin: 0;
  }
  
  html,
  body { 
      height: 100vh;
  }
  
  body {
      display: flex;
      align-items: center;
      justify-content: center;
  }
  

  Marked as helpful

  0