Design comparison
Solution retrospective
I am happy I was able to do this without looking at anybody else's work.
What challenges did you encounter, and how did you overcome them?I was confused on how to go about the whole thing at first, but I got over it.
What specific areas of your project would you like help with?None, thank you
Community feedback
- @danielmrz-devPosted 6 months ago
Hello there!
Congrats on completing the challenge! ✅
Your solution looks great!
📌 It's recommended to use semantic HTML elements like
<ul>
and<li>
for creating lists. This ensures that your code is more accessible, maintainable, and semantically meaningful.Here's and example on how you can refactor your code:
After Refactoring
<ul class="list-container"> <li><a href="#">Github</a></li> <li><a href="#">Frontend Mentor</a></li> <li><a href="#">LinkedIn</a></li> ... </ul>
By using
<ul>
and<li>
, you convey the structure of your content more clearly, making it easier for screen readers and search engines to understand. Additionally, it aligns with best practices for HTML semantics.I hope you find this helpful!
Keep up the excellent work!
Marked as helpful0 - @JamesWallison1Posted 6 months ago
Seems you have made a huge progess! Congratulation on completing the challenges! Really impressive! But just wanna have a question: Just wonder why you style the container to have cursor: pointer? I don't think this is a necessary one, I think we just need to add cursor: pointer to those buttons and those attributions, so you can consider removing them! That's all! Thanks a lot for spending time reading my comment!
Marked as helpful0
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