Submitted 11 months ago

Results summary component solution

ElijahPoblete• 20

@ElijahPoblete

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

Results summary component

HTML
CSS

1newbie

View challenge

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• 650

@Kulyk-Volodymyr

Posted 11 months 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

Please log in to post a comment

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