Responsive landing page using CSS Grid and Flexbox

Community feedback

• 

  Adriano•38,060

  @AdrianoEscarabotePosted almost 2 years ago

  Hi Josh, how are you? Welcome to the front-end mentor community! I really liked the result of your project, but I have some tips that I think you will enjoy:

  If we see how the layout is behaving at higher resolutions, with the help of google dev tools, we will see that it is stretching a lot, to solve this we can use a max-width with the value we want the content to stop growing and to center use a margin: 0 auto;

  body {
      position: relative;
      max-width: 1440px;
      margin: 0 auto;
  }
  header {
      position: absolute;
  }
  

  I made this change to the header tag to improve its behavior at higher resolutions!

  The rest is great!

  I hope it helps... 👍

  Marked as helpful

  1

  ---

  

  Josh•320

  @Ao-chiPosted almost 2 years ago

  Hi Adriano, thanks for the warm welcome and also for an awesome tips! I tried it on the dev tools and it really is stretching a lot. Will add your suggestion on my code thanks again!

  1