Skip to content
Submitted 7 months ago

Dev5

Caiquedev123•80
@Caiquedev123
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?

eu achei que esse que ia ser o mais dificil

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

deixar em coluna

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

nada

Community feedback

  • Jordan Davenport•970
    @jjdavenportPosted 7 months ago

    HTML is very well structured and easily readable as is the CSS although I would combine the media query onto a single stylesheet. The desktop and mobile layouts look good however both have problems. The desktop layout only shows for larger screens around 1440px so for most desktop screen the mobile layout is seen. The mobile layout looks good but is not contained under 250px and has issues with overflow because of this. The best way to improve the layout is to use Grid on the desktop layout for the cards to avoid unnecessary complexity.

    .card { display: grid; grid-template-rows: repeat(4, 1fr); grid-template-columns: repeat(3, 1fr); }

    The design does not differ drastically in terms of how it looks, the main styling difference is the font-weights.

    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