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

3 column preview card component with Flexbox

mihai2537 190

@mihai3636

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

P

@Islandstone89

Posted

Good job. Here are some suggestions:

HTML:

  • Change <article> to <div>.

  • I would use <img> for the icons. When you have them as inline SVGs, you must add aria-hidden="true" and focusable="false", so screen readers can ignore them. The reason we want them to be ignored is that they are purely decorative.

CSS:

  • Including a CSS Reset at the top is good practice.

  • Do not change the font size on html.

  • Remove the width on the individual cards.

  • Add a max-width in rem on the card container, so it doesn't stretch too wide on big screens.

  • Media queries should be in rem, and it is common to do mobile styles as the default. I would use Grid for this challenge, as we want 3 equal columns on larger screens.

Marked as helpful

1

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