Design comparison
Solution retrospective
I accept tips and suggestions of good practices to grow in knowledge and help the younger ones.
Community feedback
- @Nadine-GreenPosted almost 2 years ago
HEY HILVERTON!
To fix your accessibility warning, instead of using a
div
for the<section class="card">
, you could opt for a more semantic element likemain
instead, this will not only make your page more accessible, but you will also have one less error.HOPE I COULD BE OF HELP :)
HAPPY CODING!
Marked as helpful1 - @HassiaiPosted almost 2 years ago
Replace <section class="card"> with the main tag to fix the accessibility issue. for more on semantic html visit https://web.dev/learn/html/semantic-html/
There is no need to give html a width value and a height value , add the font-size and font-family to the body instead of the html.
There is no need to give the body a width of 100%, replace height: 100% with min-height:100vh; to center .card on the page.
Hope am Helpful. HAPPY CODING
1
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