Skip to content
Submitted over 1 year ago

Results summary component with simple JS

Adam Michoń•20
@AdamJou
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

On the mobile version my component is larger than the viewport so it causes a scroll bar to appear and I don't know how to fix it.

Community feedback

  • Cuong Ly•180
    @cuonglyyPosted over 1 year ago

    Hello there!

    Congrats on finishing this project! Here are a couple of things I would recommend:

    • Use rem or em instead of px for your values as they are scalable units that provide accessibility.
    • It seems like min-height: 450px in your .wrapper class is causing your results component to add extra space at the bottom.
    • Since mobile design is a smaller view, resizing some components to a smaller size may help
    • Keeping display: grid in body element for your mobile view can help center the component

    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