html css

Community feedback

• 

  Melwyn•250

  @melwyntPosted over 2 years ago

  Hi Erick! 👋 Congrats for your first challenge! 🥳

  There's only one minor issue in your report: Page should contain a level-one heading.

  You can easily fix this by replacing the h2 tag with an h1 tag.

  In your CSS, try to replace all the pixel units with rem unit. This will improve accessibility if the user is zooming the page in or out.

  You could for example add this in your CSS:

  html {
    box-sizing: border-box;
    font-size: 62.5%; // 1rem = 10px, 10px/16px = 62.5%
  }
  

  Then, instead of using font-size: 15px;, you can do:

  font-size: 1.5rem;
  

  For more information regarding this trick, this stackoverflow answer is perfect: stackoverflow.com/a/58421428/7246315

  Cheers and happy coding!

  Marked as helpful

  0