Design comparison
SolutionDesign
Community feedback
- @ABBA-74Posted over 1 year ago
Nice work, keep it up !!!
Marked as helpful0 - @ecemgoPosted over 1 year ago
Some recommendations regarding your code that could be of interest to you.
HTML
In order to fix the accessibility issues:
- You need to replace
<div class="card">
with the<main class="card">
tag. You'd better use Semantic HTML, and you can also reach more information about it from Using Semantic HTML Tags Correctly.
After committing the changes on GitHub and you need to deploy it as a live site. Finally, you should click generate a new report on this solution page to clear the warnings.
CSS
- If you want, you can use the recommended color for the background in
body
:
body { background-color: hsl(212, 45%, 89%); }
Hope I am helpful. :)
0 - You need to replace
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