Design comparison
SolutionDesign
Community feedback
- @dayumsamPosted over 2 years ago
It looks great!
Try using semantic tags: If you look at your report there are 4 accessibility issues almost all of them are because you lack a semantic tag
Solution: change your card div from a div to a <main> that should fix it.
Also can also change the font in your document to the one in the style guide
Keep it up!
Marked as helpful0 - @DjboundawPosted over 2 years ago
Thanks for advices, I have changed the div to main and the last div tags too to a footer.
Adding fonts was so difficult but using the embed link that seems easy.
0
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