Skip to content
Submitted over 4 years ago

four card section using flexbox bem scss

Sahas saurav•745
@SahasSaurav
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

any kind of recommendation is accepted

Community feedback

  • Matt Studdert•13,611
    @mattstuddertPosted over 4 years ago

    Nice work on this challenge, Sahas! As Douglas mentioned, your page isn't scrolling. This is because you've got overflow: hidden; on the .container, which should be removed. Also, you're using a div with a class of .l-heading for the main heading. A h1 would be a much better element to use here as it actually adds to the content hierarchy, whereas the div element is just a generic structural element.

    Keep up the great work!

    1
  • Douglas Varollo•470
    @DouglasVarolloPosted over 4 years ago

    In web version, your page not scrolling.

    1

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