Skip to content
Submitted 8 months ago

using display flex and grid

Depaulaeduardo•90
@Depaulaeduardo
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

What are you most proud of, and what would you do differently next time?

I think managing the display flex and grid

What challenges did you encounter, and how did you overcome them?

Trying to span the cells on display grid, It was hard to find the right cells to span it

What specific areas of your project would you like help with?

I am already working on it, display grid and flex

Community feedback

  • Adann Jacinto•790
    @DanCodeCraftPosted 8 months ago

    Well done, @Depaulaeduardo!

    You inverted the order of the cards in the middle. To avoid this, it's generally advised to work on the mobile version first, so the order is always the right one.

    I can also point out that you could consider changing your media query. It creates a weird layout in between screen sizes (the cards are not flexing, overlapping the screen, and the text in the middle becomes a huge pillar of letters. If I was seeing this on a tablet, I wouldn't be able to understand the content.

    Some points you should consider on your CSS:

    • A modern reset, so it fits every browser out there.
    • Researching different CSS measure units, to avoid issues on bigger projects - the font size is a good example: use rem, never pixel.
    • Try to use more descriptive names in your classes. If you open this same project 2 years from now, would you be able to understand it right away?

    Keep up the good work!

    Marked as helpful
    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