Space tourism multi-page website

Community feedback

• 

  Jesse Ohanugo•110

  @J-e-s-sePosted over 2 years ago

  Congratulations on completing this challenge! You did a great job. I'll share a tip I learned from completing this challenge.

  I suggest you should use "min-height: 100vh" instead of "height: 100vh" in the "h-screen" class. This allows your page content to have a minimum height of the viewport height as well as grow if the page content is longer than the viewport height. "height: 100vh" makes your page content fixed to the viewport height

  Here's what I mean:

  .h-screen { height: 100vh; // good min-height: 100vh; // better }

  This blog may help you understand better. https://greggod.medium.com/css-do-not-put-height-100-on-the-body-html-e36bda3551b3

  0