Melvin Aguilar 🧑🏻💻• 61,220
@MelvinAguilar
Posted
Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- Use semantic elements such as
<main>
and<footer>
to improve accessibility and organization of your page.
- The issue I see is that the height of the
.page
element is too small; you should usemin-height: 100vh
to cover 100% of the device height and remove theheight: 800px;
. Additionally, consider addingjustify-content: center
in the.page
element to vertically center the element instead of usingmargin-top: 180px
.
Happy coding!
Marked as helpful
0
Saleiux• 250
@Saleiux
Posted
Thankyou for your suggestions!@MelvinAguilar
1