Skip to content
Submitted over 3 years ago

HTML and CSS

Shofiya•60
@Shofiya2003
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...3-column preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hey, This is my first frontend mentor challenge. I am new to web development. Any suggestion and advice would be embraced. Please give feedback on how can I improve my code quality.

Community feedback

  • dewslyse•3,025
    @dewslysePosted over 3 years ago

    Hello Shofiya 👋! Congrats on completing your first FEM challenge👏👏. Your solution to the challenge look good and is responsive.

    A few things to note:

    • Rather than use only divs throughout your html, you could use some more specific sectioning elements. For example, your could wrap your main content on the page using the main tag. This would resolve some of the issues raised in the report.
    • Best practice dictates having one h1 on a page.
    • You could have your third card span the width of the top two between screen sizes 612px and 900px. I think that would be more visually appealing.

    All the best.

    Marked as helpful
    1
    Shofiya•60
    @Shofiya2003Posted over 3 years ago

    @dewslyse Thank you. Will note all the points.

    0

Please log in to post a comment

Log in with GitHub

Join 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