Order-summary-component with HTML and CSS (flexbox)

Community feedback

• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted about 3 years ago

  • use classes instead of id's. Classes are reusable
  • use the div with the class of card as the main element to fix some of the issues raised in the report.
  • let alt values be descriptive to benefit assistive tech users to visualize the message that is being put across. <img src="./images/icon-music.svg" alt="icon"> , screen readers automatically announce the presence of an image, writing it in the alt will be just like repetition. Decorative images use alt=""
  • well done with the presentation, it looks good

  0

  ---

  

  mcz•95

  @mauCZPosted about 3 years ago

  @ChamuMutezva Thanks for the tips and your time, i updated the deployment following the tips.

  0
• 

  Sohail Jafri•350

  @thesohailjafriPosted about 3 years ago

  GREAT WORK,

  here is some feedback for you

  1. make font-weight of 'order summary' bold
  2. add padding-bottom to the text below ' order summary'

  else everything is great🙌.

  have a good day maucz :]

  0

  ---

  

  mcz•95

  @mauCZPosted about 3 years ago

  @thesohailjafri Thanks dude, i updated the deployment following all tips :p

  0
• 

  Hafizan Adli•1,190

  @hafizanadliPosted about 3 years ago

  the annual plan card seems too tight in some mobile screen. You can make the text or padding size smaller when in small screen.

  Great job!

  0

  ---

  

  Harsha Vardhan•0

  @hackwithharshaPosted about 3 years ago

  @hafizanadli How to check in different mobile screens. is it using dev tools or there is any alternative ?

  0

  

  Hafizan Adli•1,190

  @hafizanadliPosted about 3 years ago

  @hackwithharsha in chrome, there is toogle device toolbar

  0

  

  mcz•95

  @mauCZPosted about 3 years ago

  @hafizanadli Thanks for the feedback, yeah i already updated the project deployment following the tips on the comments.

  1