stats preview using HTML CSS

Community feedback

• 

  Amon•2,560

  @A-amonPosted over 3 years ago

  Hello! Great job~ The responsiveness looks good. Aside from what's been suggested above, I would suggest:

  1. Try using picture tag for .desktop-img and .mobile-img. It lets you switch between both pictures responsively.

  2. Your design pics are not in your github .repo. Hence, it doesn't show up on the readme section.

  Two solutions submitted in one day- Keep up the efforts! :)

  Marked as helpful

  0

  ---

  

  Anitha•60

  @Anitha-vpkPosted over 3 years ago

  @A-amon Thank you :) I'm new to Front End and these suggestions really helped a lot.

  0

  

  Amon•2,560

  @A-amonPosted over 3 years ago

  @Anitha-vpk Glad to help! And... welcome onboard~

  0
• 

  Ediongsenyene Joseph I.•1,675

  @IEdiongPosted over 3 years ago

  Nice one 👍

  Some suggestions:

  • change the font-family on the <h3> to Inter
  • use <h1> in place of the <h3>. This is to solve one of the accessibility issues
  • use semantic tags like <main>, <footer>, <article>, <section>, <ul> etc to wrap around your elements, defining their meaning
  • add some spacing around the .card, it's too close to the edge at smaller screen sizes
  • work on the design at 800px screen size
  • lastly, look into the other accessibility issues raised above in the report.

  I hope this was helpful, Cheers 🥂

  Marked as helpful

  0

  ---

  

  Anitha•60

  @Anitha-vpkPosted over 3 years ago

  @IEdiong Thank you so much for the feedback

  0

  

  Ediongsenyene Joseph I.•1,675

  @IEdiongPosted over 3 years ago

  @Anitha-vpk I'm glad I could be of help 🤝

  0