HTML using CSS Grid, Flex and SASS

Community feedback

• 

  hardy•3,660

  @hardy333Posted about 3 years ago

  Hey, just use less intensive box-shadows ...

  Marked as helpful

  3
• 

  Mohamed ELIDRISSI•455

  @elidrissidevPosted about 3 years ago

  Great work overall, like @hardy mentioned just try to use less intensive shadows by increasing alpha value in shadow color and increasing the blur. Additionally I have a couple of suggestions:

  • The "Why Us" section is a list of reasons to choose this fictional service. So it makes sense to use a more semantic element, ul. It looks ugly by default but you can always style it according to the design.
  • In your CSS, I see you're using a mix of a responsive unit (rem) and absolute unit (px) which is something you'd want to avoid because absolute units do not scale with the browser's font size setting, you can learn more about responsive units in this article.

  Good luck, and keep up the good work!

  Marked as helpful

  1

  ---

  

  deenka•70

  @nomatter-pyPosted about 3 years ago

  @elidrissidev thanks for pieces of advice

  0