Skip to content
Submitted about 1 year ago

Results summary component solution

ElijahPoblete•20
@ElijahPoblete
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 find it difficult on where to host a the live website but i ended up using github i am quite happy on how this turned out, as mentioned in my last project i stopped using fixed height and width to make it responsive. i tried my best in copying the picture. please tell me where i can improve or any tips in best practices on my code thanks.

Community feedback

  • Kulyk-Volodymyr•660
    @Kulyk-VolodymyrPosted about 1 year ago

    I have looked at your code and have the following thoughts:

    • When I see the horizontal mobile version, the component is partially hidden. This can be fixed by changing height to min-height in .body.

    • Inside style-guide.md are font family properties. You can go by link and choise fonts. After that chise HTML or CSS link and place it inside code.

    • I would like to remove <button> from ul and place it below ul.

    • You can replace attribution styles from HTML to CSS.

    Good luck in the next challenges!

    Marked as helpful
    1

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