Submitted over 2 years ago
3-column-preview-card-component-main made by HTML5 & CSS
@lovishduggal
Design comparison
SolutionDesign
Solution retrospective
Please share your all suggestions
Community feedback
- @ritesh-karankalPosted over 2 years ago
I like the way you went about the project! Your solution looks great 😎
You can improve it by considering these points:
- For accessibility, The <main> element behaves like the main landmark role in screen readers. You can use the <main> element in the document for good practices.
- You may want to look at the usage of id. Use classes instead of id for classes like car-image, heading, and description.
- I noticed you have used H3 inside of a paragraph element. I would recommend you to just use <p> for description.
- You forgot to add a background color.
- There is a slight overuse of the <div> element where you could use other semantic HTML (article, section) elements also for leaner code.
keep up the good work and keep improving your skills 🛠️
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