Skip to content
Submitted 7 months ago

Four card feature section

woojjajja•90
@woohyodong
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Four card feature section
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

:)

What challenges did you encounter, and how did you overcome them?

:)

What specific areas of your project would you like help with?

:)

Community feedback

  • yinnie•280
    @wcyin9Posted 7 months ago

    Hi there, really great work!

    One thing I would suggest is implementing main in your html code for ARIA landmark purposes. It would really help the screen readers know where they are on the page. There's also no need to give header a class of header since you can just target header selector directly in CSS.

    In terms of headings, it's best practice to use h1 to h6 in order. I noticed that you started with h3, so you can use h1 and h2 instead of div first, before jumping to h3.

    You did a very good job, keep up the great work!

    Marked as helpful
    0
  • Younes Bousfiha•60
    @CoderyounesPosted 7 months ago

    Nice Work , look Awesome

    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