Responsive Stats Preview Card Component using Flexbox
Design comparison
Solution retrospective
Any feedback is welcome!
Community feedback
- @pikapikamartPosted over 3 years ago
Hey, good work on this one. The layout in desktop is good, but why did you not make any mobile state? Okay, whenever we submit our solution, always make sure to complete it. So that other could give or get some notes about it completely. Adding that mobile layout will be really awesome.
Suggestion would be:
- Please refrain from using
height: vh
units okay, since by doing so, you are only limiting the element's height based on the remaining viewport's height. Instead of this, addmin-height: vh
unit, that will make sure your content gets at least the whole viewport. Then add paddings to the top and bottom of thebody
tag, to prevent your content touching the flooring and ceiling.
Only that one. But if you were to refactor, try creating the mobile state now okay.
But still, good job^
1@rizwanmustafaPosted over 3 years ago@pikamart Thanks for the suggestions. I am sorry for the inconvenience but I had designed for mobile, although the design isn't perfect and has some problems. So I will try to reactor the code to include said suggestions and a better mobile design. Thanks
0 - Please refrain from using
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