@rhythmsaha
Posted
Thanks..
Looking to hire developers?
suggestions ?
@rhythmsaha
Posted
Thanks..
@zineb-Bou
Posted
Your site looks good on different screen sizes.
Overall semantically also looks good, just be careful you don't need to include more than <h1/>. web page has only one <h1/>.
Add text to social media links and make it sr-only
, visually hidden, and read-only by screen readers. hide the icons because we don't want the screen reader to read the image source path, the same thing applied to the hamburger menu.
Footer nav, I think they are all links, not just list items.
Any image that is purely decorative hides it from the screen reader using aria-hidden: true
.
Happy coding
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