Skip to content
Submitted about 4 years ago

3-column card with Flex, BEM, HTML5, CSS3

Art•420
@Art-wdt
A solution to the 3-column preview card component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Hey! This is my new work. I would be glad to any advice:) Any suggestions you might have! Any comments!

Feedback definitely appreciated

Code

Please log in to post a comment

Log in

Community feedback

  • minthing•145
    @minthing

    container is not centerd (if the display is bigger than 1552) how about use justify-content: center and remove margin:50px in wrapper class?

    nice work! cheer! ✨

  • ApplePieGiraffe•30,525
    @ApplePieGiraffe

    Hey there, Art! 👋

    Well done on another challenge! 👍 Your solution looks good! 😀

    I'd like to suggest using a less-important heading tags (such as <h2> or <h3>) for the titles of the sections of the card since it's generally considered a best practice to only use a single <h1> tag per page.

    Also, perhaps consider centering things with flexbox (as opposed to margin or padding). In this case, it would make centering the card component in the viewport quite easy, I think! 😉

    Keep coding (and happy coding, too)! 😁

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