Responsive Stats Preview Card Component using Flexbox

Community feedback

• 

  Raymart Pamplona•16,040

  @pikapikamart

  Posted about 4 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^

  11