Design comparison
Community feedback
- @SamadeenPosted over 2 years ago
Hey!! Cheers 🥂 on completing this challenge.. .
Here are my suggestions..
- You should use <main class="card"> instead of <div class="card">.
- Go down orderly when you are using the headings h1 down to h2 down to h3 and so on.
- You should add a
alt
text to yourimg
tag to aid screen-readers
This should fix most of your accessibility issues
. Regardless you did amazing... hope you find this useful... Happy coding!!!
Marked as helpful0 - @christian-prasetyaPosted over 2 years ago
Hello if I may helping.
First of all, cheers of finishing this challenge.
But I think you should change the
<div class="card">
to<main class="card">
and adding a alt to your<img>
to help the issue with your code submission. Believe me, I did the same mistake before, and secondly, it will help you in the future to fix the semantic and accesibility of your website in the future.Hope this will help.
Cheers.
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