bookmark landing page

Community feedback

• 

  Chris•240

  @ChristopherParkePosted over 2 years ago

  Not too bad. I see from the screenshot that the footer is not quite sticky. You can fix this by making the body have height of 100vh. I also like to make the body display as a grid. I do this because I like to make the page layout with grid since it's 2 dimensional, instead of 1 (like flexbox).

  body {
       display: grid;
       height: 100vh;
  }
  

  Also, your mobile design is quite off. You should inspect your page with the "device toolbar" active in the dev tools. You will see that you have a side scrollbar and things aren't the right size or alignment.

  Marked as helpful

  0

  ---

  

  Sourabh•420

  @SOURABH358Posted over 2 years ago

  @ChristopherParke Thanks for your review. Some last moment changes messed up the mobile design. Thanks for reviewing. 👍

  0