Design comparison
SolutionDesign
Community feedback
- @anoshaahmedPosted almost 3 years ago
To get rid of the accessibility/HTML issues shown in your Report:
- wrap everything in your body in
<main>... OR use semantic tags! .... you can also giverole=""to the direct children of your<body>but that's a little frowned upon... Click here to read more - have at least one
<h1>in your code - add
alt=""to your<img>tags <section>and<article>usually need a heading; so if you don't need a heading in it, use some other element such as<div>
If you use the same border radius on your image, as the one you did on the card, it would look more flattering.
Great job on your first project Eric! :)
Marked as helpful0 - wrap everything in your body in
Please log in to post a comment
Log in with GitHub
Join 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