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

(My first challenge) Responsive Stats preview card component

@DiegoVelazquez60

Desktop design screenshot for the Stats preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi! This is my first challenge in frontend mentor. I would love if anyone could give me any feedback or if I could do something to the CSS code and make it look less messy. :)

Community feedback

Account Deleted

Hey @DiegoVelazquez60,

Good job on your first challenge! Here are a couple of tips to help improve your solution until next time

  • Multiple elements should never share the same ID attribute. Make sure that you use classes instead, as specified in the report. Read more about the differences between IDs and classes here
  • The image on desktop version in your solution seems a bit distorted. The issue at hand is that your section#text has too much margin space, causing the right side to increase in height as well. To solve this, I would suggest removing the text id from your <p> tag on line 25 in your index.html file

Good luck on your next challenge 👍

//Kenny

1

@DiegoVelazquez60

Posted

@ktra99 Hi! Thanks for the feedback

I honestly did not notice the IDs until the report detected it 😅; but thank you very much for your comments and I will do what you say in the line 25. :)

1

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