grid flex-box css3 html-5 etc..

Community feedback

• 

  Muhammad Shajjar•1,100

  @muhammadshajjarPosted about 3 years ago

  Hey, awesome work on this one. Layout in general looks really great. I have some feedback on your solution.

  • Try to add max-width to your body or make a container class which wraps your all content, by adding it you can control your content from growing too much on larger viewports
  • On patrick testimonial try to add this background-color:hsl(217deg 30% 14%)
  • Give your body to background-color:hsl(210deg 46% 95%)
  • use rem or sometimes em instead of using px because the size of the pixel isn't always the same.

  0
• Account deleted

  Hello rayir03. Imylooking your solution in my mobile phone and it’s not responsive. You must fix it first. Also accessibility issue

  0