Skip to content
Submitted about 1 year ago

Result page

sickopatricko•10
@sickopatricko
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Results summary component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Im definitely sure that i did the height for desktop media query the wrong way and it could be done better. Someties i have no idea when to use flexbox or grid so i eventually end up with using flexbox most of the time.

Community feedback

  • Kirsten ✨•460
    @ofthewildfirePosted about 1 year ago

    Hi! Your solution look really great! :)

    I only have one suggestion, your .result and .summary on desktop are not even, since you're using Flexbox, you can add a flex-basis: 50% to each of them so they are equal. On just the desktop version!

    PS: In case you care about the color being perfect, which is totally not needed, the button hover state is actually the gradient given for the purple card! I do prefer your color though. :)

    Your height looks fine to me as well. I also struggle not knowing when to use Flexbox or Grid, I usually end up using Grid, since I tend to like it more these days. :)

    Anyways, hope this was helpful, and happy coding!

    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