@ApplePieGiraffe
Posted
Hey there, wenadev! π
It's me again! π
It's great to see you completing so many challenges! π Well done on this one! π
A few things I'd like to suggest are,
- Turning the navigation links in the footer of the page into actual links using the anchor tag.
- Perhaps setting
pointer-events: none
to thediv
overlaying the footer of the page so that the elements in the footer can be clickable. - Adding a
max-width
to the main wrapper or container would be a good idea to prevent the content of the page from becoming too wide on extra-large screens. π
As always, keep coding (and happy coding, too)! π
@wenadev
Posted
@ApplePieGiraffe thank you for contribution! site updated! :))
@ApplePieGiraffe
Posted
@wenadev
Nice job! The updated site looks good and works well! π
Now that you've added a max-width
to the content on the page, one last step might be to allow the backgrounds of the header and footer to grow with the width of the screen (rather than be affected by the max-width) so that there is no empty white space to the sides of those sections on extra-large screens.
There are a few ways of doing this, but one way I would achieve this effect was by wrapping the content of the header and footer sections in a container and setting the max-width on the container rather than on the entire header and footer. That way, the header and footer could stretch to the ends of the page, but the container and everything inside it would remain together because of the max-width. Just a tip! π
Keep it up and have fun! π