Design comparison
Community feedback
- @AdrianoEscarabotePosted 17 days ago
Hi nabhoneel, how are you doing? I really loved the outcome of your project, but I have a few suggestions that I think might be helpful:
To enhance the semantics of your code, consider using a
<ul>
(unordered list) for the collection of links, as it represents a list of related items. Here's an example:<ul> <li><a href="#">GitHub</a></li> <li><a href="#">Frontend Mentor</a></li> <li><a href="#">LinkedIn</a></li> <li><a href="#">Twitter</a></li> <li><a href="#">Instagram</a></li> </ul>
Using a
<ul>
provides clear structure and context, signaling to both browsers and assistive technologies that these links are part of a cohesive group, improving both accessibility and readability.The rest is excellent.
I hope you find it useful. 👍
1 - @oMarcelosantosPosted 18 days ago
Good job !
-
I really liked the idea of your website being responsive. I liked it when I hovered the mouse over the buttons and they changed their color.
-
You can assign the following tag to your code so that when someone clicks on the button, they will be taken to another URL. ex:
<a href="https://www.frontendmentor.io/profile/yourprofile" target="_blank"> <button>Frontend Mentor</button> </a>
1@bluesky0-0Posted 18 days ago@oMarcelosantos Hey thanks man i will include your suggestion on the next project
1 -
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