Skip to content
Submitted over 3 years ago

Mobile-first 3 column preview card grid and flexbox

P-H•90
@Paul-henry662
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

Hi there! I tried my best to complete this challenge using grid layout associated with flexbox so any feedback regarding that is welcome. Also if you have any thought on the HTML markup, don't hesitate.

Community feedback

  • Vanza Setia•27,775
    @vanzasetiaPosted over 3 years ago

    👋Hi Paul-henry662! My name is Vanza!

    I have some feedback on this solution:

    • Replace all your h1 with h2, since it's a best practice that only have one h1 on every html page.
    • Replace all your button with a tags since if it is a real website, the learn more button will point to another page.
    • For your section h1, try change the font-size using rem unit. It's simply because it will allow the user to change the font size base on their browser setting.

    That's it! Hopefully this is helpful!

    Happy Coding!

    1
    P-H•90
    @Paul-henry662Posted over 3 years ago

    @vanzasetia Thanks a lot!

    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