Responsive page using FlexBox

Community feedback

• 

  Suleyman•360

  @SuleeymanPosted over 3 years ago

  The layout is well done, except your image between [900px - 800px]. To remain for this issue, you can simply add a height: 100% on your img__container. Then add height: 100% on your images and, object-fit: cover;

  Also, you should add a padding-top and -bottom on your flex__container--content. Moreover, you don't need a neested <div> on your flex__container--content. And for finish, in SCSS, variables are declared like:

  $my-variable: value; And you don't need to nest them in your :root element.

  In SCSS we can nest our CSS style like: header { h1 { } } Which means in CSS: header h1 { }

  0