SCSS grid and flexbox layout

Community feedback

• 

  AbdeRaouf Zemmal•680

  @abdraoufxPosted over 2 years ago

  Hey Mate, Greate Job; But I Have an Advice For This Design To Make It Better:

  • You Need To Add Some Space Between The Hero Picture And The Text.
  • Add margin From The Bottom Side For The Button To Make Space Between it and the 4 images below it.

  1

  ---

  

  Ali Ahmed•680

  @Dany-GitHubPosted over 2 years ago

  @abdraoufx i wonder why it looks so weird after i uploaded here , check README on GitHub repo it looked way better, anyway thanks for the suggestion

  0

  

  AbdeRaouf Zemmal•680

  @abdraoufxPosted over 2 years ago

  @Wa7dany I Think Because Of The Display Grid Try Change It To FlexBox In Your Developemment Envioremment And After Finish Trying You Can Check My Solution : https://www.frontendmentor.io/solutions/full-responsive-intro-section-with-dropdown-navigation-with-sass-and-js-rkMqqstN5

  0