Submitted over 1 year ago
Frontend Mentor - Testimonials grid section solution
@waad2lf
Design comparison
SolutionDesign
Solution retrospective
All feedback is welcome thank you in advance
Community feedback
- @Thewatcher13Posted over 1 year ago
-
you should have always a main landmark element included in your html
-
NEVER skip the headings order in html, there should be first an h1 before you using an h2
-
use a css reset, look on Andy's Bell website for a good and clear one
-
never use px for font-size but rem https://fedmentor.dev/posts/font-size-px/
-
your font-family should be inculde in the html, like the design says
-
dont use ID in css, bcs is has a high specificity, use classes
- I saw you're solution from the qr-code challenge, you do better other and easier projects then this, please apply the feedback that you get on your projects before start a new one.
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