Design comparison
SolutionDesign
Community feedback
- @dev-paulLPosted 10 days ago
Hi Edgar,
- Replace
class="container"
with<main>
for better structure and accessibility. - Use
alt=""
instead ofalt="illustration"
. - You could rename
div class="learning"
todiv class="chip"
or "category". - On a real website, ensure
h1
andh2
are used ifh3
is present to maintain heading hierarchy. - The
<picture>
tag is useless here, you can remove it and use <img>. - Use
alt="Greg Hooper's avatar"
oralt=""
but not alt="avatar". - Avoid
px
forfont-size
; useem
orrem
. - Make the card clickable to align with preview functionality expectations.
- Pay attention to the style guide (colors/spacing).
Happy coding
0 - 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