Responsive Stats Preview Card using CSS flexbox
Design comparison
Solution retrospective
Your feedback is highly appreciated
Community feedback
- @zambobencePosted over 2 years ago
Great job, semantically totally understandable, I really liked that you paid extra attention to the alternative text for the image.
I am not familiar to React and have never used it (currently only using HTML CSS and vanilla JS) but if I were you I would put a max-width on the "stats-card" so that it stops growing until some screen width.
In horizontal mode, the image does not reach to the bottom of the is a small purple line where the image does not reach until the bottom of the container "stats-card-img-container". You can resolve it by setting the height to 100% in the css selector "stats-card-img-container"
A small detail I have also missed in my project is, to set the correct border radiuses of the "stats-card-img-container" and image mobile and in desktop view. (mobile - top right, top-left of the image etc.)
I hope I could help with my input. :)
Marked as helpful0@BasharKhdr1992Posted over 2 years ago@benceturbulence Thank you so much for your feedback. I will go over your notes and apply them
1
Please log in to post a comment
Log in with GitHubJoin 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