P
ApplePieGiraffe• 30,545
@ApplePieGiraffe
Posted
Hey there, wenadev! 👋
It's good to see you complete another challenge! 😀 Nice job on this one! 👏
A few things I'd like to suggest are,
- Turning the social media links in the footer of the page into actual links using the anchor tag (giving them a hover state would be a nice touch, then, too).
- It appears that I can't click on and use the input element in the footer of the page because a
divis covering that entire section. You might want to addpointer-events: noneto thatdivto avoid that. - Adding a
max-widthto the main container or wrapper of the page to prevent the content of the page from stretching too much on extra-large screens. 😉
Keep coding (and happy coding, too)! 😁
Oh, and hey, if you found this comment helpful, an upvote would be appreciated! ;)
2
wenadev• 425
@wenadev
Posted
@ApplePieGiraffe thank you so much for the suggestions! site has been updated! :))
1
P
ApplePieGiraffe• 30,545
@ApplePieGiraffe
Posted
@wenadev
Just took another look at your solution, and the changes are great! Everything works nicely in the footer now, and the max-width keeps everything together. Good work! 👍
0
