Flebox, responsive design

Community feedback

• 

  Benjo Quilario•1,810

  @benjoquilarioPosted over 3 years ago

  Congrats on this finishing your 1st frontend project.

  Suggestion:

  1. Remove the height: 100vh on the body tag. Because this will only limit the body to have a height 100% of the remaining viewport or screen. 2 . Change the background color of div="flex" to body.
  2. Remove or add code on the div="container". Because you didn't any code on it basically it useless. 3.1. add margin: 0 auto; width: 1000px or any pixel you want and max-width:100%.
  3. Try add width: 100% on image.

  Aside from those, really good job on this.

  Marked as helpful

  1

  ---

  

  P

  Aleksandra Ryan•130

  @AleksandraRyanPosted over 3 years ago

  @benjoquilario Thank you for your suggestions !

  0
• 

  Akash Verma•810

  @akash-1712Posted over 3 years ago

  In mobile layout set width:initial for div.flex it allow to center the container horizontal.

  Marked as helpful

  0