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

Results summary challenge

P

@Temceo

Desktop design screenshot for the Results summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

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,230

@hitmorecode

Posted

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

P

@Temceo

Posted

Thank you @hitmorecode

0

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