Stats Preview Card Component

Community feedback

• 

  Ejovi Okiekpakpo•345

  @ruona88Posted over 3 years ago

  Hello there,

  Your solution looks good. But I noticed there are no media queries for mobile viewing.

  0

  ---

  

  Shehan•40

  @ShehanMaduwanthaPosted over 3 years ago

  @ruona88 Thank you very much for the feedback. I've included the media queries but they follow the style guide provided. So the breakpoint might be lower than the lowest width the browser can show.

  0
• 

  Kris Pietrzak•340

  @krisp-devPosted over 3 years ago

  The desktop design looks good overall, it is good practice that you have used relative units throughout your code and the code itself is pretty easy to read for a beginner like myself.

  A few minor things I'd touch up on are the font-size (a little bit smaller than the original design) and I'd play around with the filter of the image (contrast and brightness) but I am being really picky here.

  Also, I think you forgot to make it mobile responsive.

  0

  ---

  

  Shehan•40

  @ShehanMaduwanthaPosted over 3 years ago

  @krisp-dev Thank you very much for your feedback. I agree that the font size is a bit smaller than the original design. I included the media query to make it mobile responsive in the _card.scss partial.

  Thank you very much for the feedback again !

  0