HTML CSS + FLEXBOX

Community feedback

• 

  Allan Kirui•210

  @AllanKiruiPosted over 3 years ago

  Hello Paras. First off all, you should be proud of yourself for completing this challenge. Many people started this challenge, but you are among the few who have completed it.

  Here is some feedback on some of the things you might want to improve on. I suggest that you use class names that are a little bit more easy to read. For example, where you used a class of 'div div1' for a div element, I would suggest that you use a class names that try to represent the information that you are styling. Class names such as 'testimonial' or 'card card1'. This applies to the ratings section as well.

  For the user images, in the testimonial section, it's considered best practice to use width and height properties on images. Here, I suggest you consider using px/rem units to set the values of the width and height properties as well as setting margins, paddings etc. This might help you get as close to the design as possible.

  But as I said, be proud of yourself. Keep coding and never stop learning new things and improving :D

  Marked as helpful

  1