Melvin Aguilar 🧑🏻💻• 61,220
@MelvinAguilar
Posted
Hello there 👋. Good job on completing the challenge !
-
In the body/html element, neither of the two ways. The body element is a block-level element, so by default, it has 100% width, and from the projects I've seen, using
width: 100vwgenerates an unnecessary horizontal scrollbar on some screens. So you can remove it.It's not necessary to use max-width or min-width on the body element; let it take up 100% of the screen by default. The internal elements are the ones that should have max-width to prevent them from stretching too much, as you currently have it.
- You also don't need
height: 100%on the body element, as you are usingmin-height: 100vhon the<main>element, and the height won't take effect.
- The use of background is fine.
I hope you find it useful! 😄
Happy coding!
1
