Stats Preview Card using CSS Grid

Community feedback

• 

  CyrusKabir•1,885

  @CyrusKabirPosted about 3 years ago

  hello my dear friend ♥ your project it's good and clean but here some little problems :

  • try use more semantic elements like <main>,<footer>,<section>,etc.
  • the "m"&"k" in numbers should be uppercase at last everything was fine and i wish you good luck and go ahead for more challanges ♥♥♥

  Marked as helpful

  0
• 

  Juveria Dalvi•1,375

  @Juveria-DalviPosted about 3 years ago

  hey.. the accessibility issue says that you should have wrapped all divs in <main> which is the semantic tag ,except for the div with class attribution that should be wrapper in <footer> tag for more details refer w3school.com

  Marked as helpful

  0
• 

  Ted Dino•170

  @ted-dinoPosted about 3 years ago

  Hi Shivang, I'm a newbie and also learning here in Frontend Mentor I just want to share what I see on your code. I notice in your styles.css you have a multiple of @media (min-width 50em). I think your css will be more organized if you put them in one @media (min-width 50em) other than that it's nice and it also looks good on mobile. Keep up the good work and try to fix the accessibility issues. Have a great day!

  Marked as helpful

  0