Mijail Hernandez• 950
@MishaHernandez
Posted
Hello Nofar Aviv 👋
It looks great, congratulations.
I just have a little remark: in desktop layout the main component has a lot of margin-top and this can produce a scroll bar on some screen sizes. So I would recommend removing margin-top and using flexbox or grid to center it horizontally and vertically.
body { display: flex; align-items: center; height: 100vh; }
@media (min-witdh: 1200px) {
container {margin-top: 0; }
}
I hope this is helpful to you. Happy coding 🙂
0
Nofar Aviv• 120
@nofaraviv
Posted
Thank you very much for the advice! I learned something new :)
0
