@danielmrz-dev
Posted
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 helpful
@danielmrz-dev Hi Daniel! Wow, thanks a lot for your thoughtful comment :) I have now changed the implementation to use <ul>
and <li>
instead of <div>
and <button>
. I honestly had no idea it made more sense to represent this as a list so thanks for clarifying :)
Btw, your score here on Frontend Mentor is incredible! Awesome Github profile too! Keep it up :)