Skip to content
Submitted about 1 year ago

Results summary challenge

Martin Mwaka•390
@Temceo
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

I found it challenging to organise the site using flexbox, using a mobile first design. I would welcome any feedback on how use flexbox and organise my code more efficiently to avoid repetition.

Community feedback

  • hitmorecode•6,680
    @hitmorecodePosted about 1 year ago

    Nice well done. Just a few tips

    • When on mobile the card is stuck at the top of the page. If you add min-height: 100vh; on the body it will place the card in the middle of the page.
    • Also when om mobile the card stretches from side to side, you should make it at least 10 maybe 15px smaller than the width of the screen.

    You forgot to add the border radius on mobile

    Marked as helpful
    0
    Martin Mwaka•390
    @TemceoPosted about 1 year ago

    Thank you @hitmorecode

    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