Stats preview card component

Community feedback

• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted over 3 years ago

  Congradulations for finishing you first challenge.

  • at 375px and below the site looks good but immediately after the site breaks. Your breakpoint @media screen and (max-width:375px) was too early , it has to be adjusted. A user must be able to view all the content regardless of screen size
  • your css file is formated to the far right such that at first i thought the file was empty
  • use css to uppercase text , avoid this <p class="stat-head">COMPANIES</p>, screen readers read uppercased words , letter by letter (not as a word)

  Marked as helpful

  0

  ---

  

  Saya Takahashi•10

  @saya-takahashiPosted over 3 years ago

  @ChamuMutezva Thank you very much for your advice! I did not pay attention to points you mentioned at all... I'll check the css file again and revise the points. I appreciate with your comment! ;)

  0
• 

  Saif Narpali•590

  @SaifN97Posted over 3 years ago

  Hey Saya! Great work on this one!🎉

  I'll suggest look more into using the combo of rem and em units.. I struggled on this one too but learning to use the correct units helped me big time on my next projects!!

  Have fun coding :) Cheers!

  Marked as helpful

  0

  ---

  

  Saya Takahashi•10

  @saya-takahashiPosted over 3 years ago

  @SaifN97 Thank you very much for your advice! Yes... rem and em units are difficult for me to understand, but I'll try to master the function. I appreciate with your comment! :)

  0