Stats Page/HTML-CSS

Community feedback

• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 9 months ago

  Hello @jcbsmith20!

  Your solution looks excelent!

  I have just one suggestion:

  📌 Consider this tweak for your HTML code:

  To improve semantic clarity, opt for <h1> over <h3> for your main title.

  It's more than just text size — it's about structuring your content effectively:

  • <h1> to <h6> are used to define HTML headings, with <h1> being the most significant.
  • Stick to one <h1> per page for the main title, and maintain the titles hierarchy with <h1>, <h2> <h3>, and so on.

  While these adjustments might not alter the visual appearance much, they significantly enhance semantic clarity, SEO optimization, and accessibility.

  📌 Also, I had a hard time myself trying the image coloring when I did this project, but I managed to do it like this:

  .image {
      background-image: linear-gradient(0deg, #7210af9c, #7210af9c), url('images/image-header-mobile.jpg');
      background-size: cover;
      height: 240px;
  }
  

  There's also another option using a property called background-blend-mode, but I confess I don't know how to use it very well. I'm mentioning it because you might wanna look into it. 😊

  Hope it helps!

  Other than that, your project looks fantastic!

  0