Clipboard Landing Page | Responsive | HTML & SASS
Design comparison
Solution retrospective
Any feedback or suggestion will be appreciated. Thanks in advance :)
Community feedback
- @ApplePieGiraffePosted over 3 years ago
Hi there, Arif Khan! 👋
Nice to see you complete another challenge! 😀 Well done on this one! 👍 Your solution looks good and is responsive and I like the extra features you added such as the custom favicon and text that appears above the social media icons when they are hovered over! 👏
I suggest wrapping the social media icons in anchor tags (since they seem like that would be links that would take users to their respective social media site) and perhaps taking a look at your solution report and trying to clear up the error or two that is there in order to improve the semantics and accessibility of your HTML. 😉 Those are small things, though.
Keep coding (and happy coding, too)! 😁
Marked as helpful1@arif-khan-dhPosted over 3 years agoHi @ApplePieGiraffe
Thanks for the comment and your suggestions. I just fixed the HTML issues and wrapped social media icons inside anchor tags. Please keep giving us feedback. Happy coding :)
0
Please log in to post a comment
Log in with GitHubJoin 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