Responsive Stats Preview Card Component using Flexbox

Community feedback

• 

  Raymart Pamplona•16,100

  @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:

  1. 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, add min-height: vh unit, that will make sure your content gets at least the whole viewport. Then add paddings to the top and bottom of the body 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

  ---

  

  Rizwan Mustafa•155

  @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