All comments

• 

  Debraj Bhal•20

  @krishna-vasudevSubmitted about 3 years ago

  Open to your valuable suggestions and feedbacks.

  

  Responsive layout using CSS flexbox.

  1

  10
  

  YoSoyPan•0

  @yosoypanPosted about 3 years ago

  The most glaring issue I see is the line breaks not matching the reference. This can be fixed with <br> tags and defining the width of the text in question. Defining the width will ensure that your line breaks stay consistent and don't get messed up and resizing. I provided a fix in a pull request on your GitHub so you can see this in action. Overall great job Debraj!

  1

  This comment has received 1 replyShow
• 

  maym42•370

  @maym42Submitted about 3 years ago

  Hi there, I would love to get feedback on my code! 🤩

  

  stats preview card component main (mobile first)

  1

  30
  

  YoSoyPan•0

  @yosoypanPosted about 3 years ago

  I would recommend getting the exact dimensions of the container in the reference photos. You can do this with an app like Pixelzoomer or whatever you fancy. Then, define those dimensions (width, height) in the style of your main-container class and that will eliminate the stretching. Great work, keep tapping!

  Marked as helpful

  0

  This comment has received 1 replyShow