Stats Components Review

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted 11 months ago

  Hello there 👋. Good job on completing the challenge !

  I have some suggestions about your code that might interest you.

  • Wrap the page's whole main content in the <main> tag.

  • You should use only one <h1> tag per page. The <h1> tag is the most important heading tag, This can confuse screen reader users and search engines. You can read more about this here 📘.

  • The <center> tag is considered obsolete in HTML5, and it is recommended to use CSS styles for centering elements. Two modern and flexible approaches for centering elements are Flexbox and Grid.

    Using Flexbox:

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

    Using Grid:

    body {
      min-height: 100vh;
      display: grid;
      place-items: center;
    }
    

  • Utilize a CSS reset to eliminate default styling variations across browsers.

    Popular CSS resets:

    • Reset CSS
    • "My Custom CSS Reset" by JoshWComeau

  I hope you find it useful! 😄 Above all, the solution you submitted is great!

  Happy coding!

  0