Skip to content
OverviewSolutions
3
Comments
1
Ritesh Karankal
@ritesh-karankal

All comments

  • Ritesh Karankal•50
    @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 helpful
    0