Design comparison
Community feedback
- @mo-rob-1Posted almost 3 years ago
Hey! You did a great job on this challenge. Well done.
After looking at your code, a tip I have is to have a look at the BEM methodology (Block Element Modifier) for naming your classes. This helps to keep your code organized and maintainable E.g. You could have a container div named 'card' and inside that div you can use naming conventions like 'card__img', 'card__desc' etc for the image and text.
Hope this helps & keep up the good work :)
Marked as helpful0 - @NaveenGumastePosted almost 3 years ago
Hay ! Larissa Cauane Good Job on challenge
These below mentioned tricks will help you remove any Accessibility Issues
-> Add Main tag after body
<main class="container"></main>
-> Learn more on accessibility issues
If this comment helps you then pls mark it as helpful!
Have a good day and keep coding 👍!
Marked as helpful0 - @EmmanuelHexerPosted almost 3 years ago
Great work overall man. Nicely done
- One tip i can give your is to wrap your card with main element to avoid accessibility issues.
0
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