Stats preview card component using HTML5 and CSS3

Community feedback

• 

  Mojtaba Mosavi•3,760

  @MojtabaMosaviPosted almost 3 years ago

  1- Desktop and mobile view behave properly but there are some minor overflowing and overlapping issues main because of spacig of the stats is dependent of a flex property and padding being set using presentages, use some like rems to see the differece in behavior.

  2- stats could be implemented using an unordered list.

  Keep coding :=)

  Marked as helpful

  0
• 

  Omar Mohy•670

  @omarmohy98Posted almost 3 years ago

  Hi@eyagargah, wonderfully done! The code is remarkably responsive. Below is my feedback on potential improvements!

  1.) Instead of <div class="card"> , let's replace that with <main> element. Doing so will give your code more semantic meaning, will allow screen readers to know this is your main content and Fix the issues appear in Accessibility Report.

  2.) All of your images should have alt attribute with content within the quotations.

  If you found my feedback helpful, please mark this comment as helpful. Wonderfully done. Happy coding!

  Marked as helpful

  0

  ---

  

  Eya Gargah•230

  @eyagargahPosted almost 3 years ago

  @omarmohy22 thank you so much for taking the time to comment great feedback I appreciate the helpful advice!

  0

  

  Omar Mohy•670

  @omarmohy98Posted almost 3 years ago

  @eyagargah you are welcome

  0