@jakegodsall
Posted
Hi 👋
Great work on this page. It looks great 😁
I'd suggest adding a max-width
and centring on the main container of the page because on wide viewports the <header>
component looks very spread out.
With this challenge its not too much of a problem because the content of the page is really all centred but it can become a bit of a problem when the entire content of the page is spread across thousands of pixels. I tend to use a max-width: 1600px
or something around that size.
A nice addition if you're looking to expand the project would to be to add expand/collapse animations for the dropdown menus in the mobile menu. An example of what I mean can be seen in the following codepen.
Hope this helps 😁
Marked as helpful
@AhmedHamdy146
Posted
thank you @jakegodsall