Nft Card com elementos de HTML e CSS grid.

Community feedback

• 

  Marcos Travaglini•4,840

  @BlackpachamamePosted 10 months ago

  Greetings! you have done a great job 😎

  📌 Some accessibility and semantics recommendations for your HTML

  • To improve the semantics of your HTML, you can change your <div class="container"> to a <main class="container">
  • The attribution should go outside the main

  📌 Some suggestions

  • I recommend doing a small reset to the styles that come by default in the browsers. To do this, you can apply a couple of properties to the universal selector *, with this you will make your site look the same in all browsers
  • I leave you the task of researching the reset CSS and the box-sizing: border-box
  • If you didn't apply the reset, you can add margin: 0 to your body, this will remove annoying scrolling on large screens. If you want to maintain separation on very small screens, you can apply the margin again using media querys
  • You can use the gap property in the body to separate your main from your footer
  • Instead of using margin to center your content in the center of the screen, you can use the flexbox properties in the body:

  body {
      background-color: hsl(217, 54%, 11%);
      /* padding: 100px; */
      font-family: 'Outfit';
      margin: 0; /* reset */
      min-height: 100vh;
      display: flex;
      justify-content: center;
      align-items: center;
      flex-direction: column;
      gap: 20px; /* Separate the main from the footer */
  }
  

  Marked as helpful

  1