Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Space tourism multi-page website

Carolina 290

@CaroBilkis

Desktop design screenshot for the Space tourism multi-page website coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
3intermediate
View challenge

Design comparison


SolutionDesign

Solution retrospective


Any feedback is welcome. Happy coding!

Community feedback

@J-e-s-se

Posted

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

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord