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-Component-Main

@AnderCMD

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


just finished the project, I would appreciate any feedback to improve, excellent day everyone! ✅💻

Community feedback

@0xabdulkhaliq

Posted

Hello there 👋. Congratulations on successfully completing the challenge! 🎉

  • I have other recommendations regarding your code that I believe will be of great interest to you.

BODY MEASUREMENTS 📐:

  • The width: 100vw property for body is not necessary. because it's a block level element which will take the full width of the page by default.
  • So feel free to remove width: 100vw style rule from body this will help you to write efficient code and makes your code more reusable.

.

I hope you find this helpful 😄 Above all, the solution you submitted is great !

Happy coding!

Marked as helpful

1

@AnderCMD

Posted

@0xAbdulKhalid Thank you for your feedback, I will take it into account and modify it shortly.

0
Luka Glonti 3,440

@lack21

Posted

Good work 👍, but I have some recommendation!

  • Remove width: 100vw from the body, since body by default takes the maximum width available, so there's no need for width: 100vw.

Marked as helpful

1

@AnderCMD

Posted

@lack21 Thank you for your feedback, I will take it into account.

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