Design comparison
Community feedback
- @patriciarrsPosted 3 months ago
Greetings, musaabuu! 👋
Good effort on this challenge! 👍 The layout looks good on a range of screen sizes 🙂
I want to suggest taking another look at using semantic HTML on your site as it will also improve accessibility. Currently, you are using
div
tags to wrap other elements, but other tags (for example,main
andnav
) are more suitable.If you’d like to learn more about how to use semantic HTML on your site, check out this course from web.dev.
You may also consider using the
:focus
pseudo-class so that, when a user navigates the page with the keyboard, he gets the same experience as if he were hovering the links.I hope you find this helpful.
Happy coding!
Marked as helpful1
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