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

HTML and CSS / Mobile First, BEM, Flexbox

@o0oDanielHaroo0o

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


This is the first time I attempt to do a responsive page, I'm still kinda lost so I welcome any advice/comment/critique. Thanks

Community feedback

Mathieu 295

@mathcrln

Posted

Hello Daniel, well done!

I don't have much to say, your design is very well implemented and especially smooth. Nicely done for the first time! Additionally, I checked out your code and it's very easy and agreeable to read with the spacing and the comments. It seems clean and simple, yet effective. Congrats 👏🏾

When it comes to feedbacks, around 1125 pixels, things can seem a bit constrained and can seem to "jump" at times. I see that you added varying padding on the .card__content class. Maybe instead you could a div for the information and rather play with the max-width to get the effect you want?

Between 700 and 1140 pixels, the title and the paragraph are very close touching, maybe you could improve the spacing on these elements.

Otherwise, well done! Good luck for your upcoming projects and challenges.

Marked as helpful

2

@o0oDanielHaroo0o

Posted

@mathcrln Hi Mathieu, thank you!

I see what you're saying about the padding, I'll try that. I'll fix the spacing as well. Thanks for the feedback!

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