Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive four card feature selection with additional layout

relizard 80

@RestlessLizard

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hey community, could you please review my solution? I'm very interested in what can be improved visually and especially code-wise (I'm using BEM with minor class composition). Thank you!

Community feedback

Adriano 34,090

@AdrianoEscarabote

Posted

Hi relizard, how are you?

I really liked the result of your project, but I have some tips that I think you will enjoy:

As you know this project is based only on one page section, so no h1 is needed! because we don't know if this page will have a more important component, but it's always good to prevent accessibility errors so I think it would be good for you to add an h1 in this component, besides being a good practice for when you are developing larger sites, don't worry forget about h1.

The rest is great!

I hope it helps... 👍

Marked as helpful

0
a13g24 40

@a13g24

Posted

I recently implemented this design as well, and I really learned a lot from your implementation. Especially with the use of margin-block, something I was unfamiliar with until reading your code.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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