Design comparison
Community feedback
- @akshaywebsterPosted almost 3 years ago
Hi, Florin.
Nice work with this project. Keep it up, mate!
To get rid of the Accessibility issues:
-
Make sure you have one h1 tag in your HTML document.
-
Your HTML document should have one main landmark. You can achieve this either by using semantic HTML tags or by declaring your main div with role="main".
It's always a great idea to learn about HTML5 semantic tags to reduce errors like this as well as to make your site more readable by screen readers.
Here's a great video on that: HTML5 Semantics
- Also, your card is not vertically centered, so please look into that if you have some free time.
Overall, you've done a great job since this is your first project here on FrontEnd Mentor.
Hope my feedback is helpful to you. Have a great day, Florin! :)
Marked as helpful0 -
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