Folarin Akinloye• 1,240
@folathecoder
Posted
Thank you very much!
I appreciate your feedback!
1
Submitted
Desktop-First & Responsive: HTML, SCSS, FLEXBOX, SVG MANIPULATION
Folarin Akinloye• 1,240
@folathecoder
Design comparison
Solution retrospective
Hi Legends!
Please check my implementation and my code!
All feedbacks are welcome!
Community feedback
P
ApplePieGiraffe• 30,545
@ApplePieGiraffe
Posted
Hey, Folarin Akinloye! 👋
Nice to see you complete another challenge! 😀 Overall, your solution looks good and responds quite well! 👏
I'd like to suggest,
- Adding
cursor: pointerto the buttons on the page. - Turning the social media icons in the footer of the page into links by wrapping each of them in an anchor tag.
- Allowing the backgrounds of the various sections of the page to fill up the entire width of the screen so that there isn't any white space to the sides of the design on extra-large screens.
- Taking a look at your solution report and trying to clear up some of the errors that are there in order to make your solution more accessible.
Of course, keep coding (and happy coding, too)! 😁
1
Please log in to post a comment
Log in with GitHub
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord