@ameyamjoshiSubmitted over 3 years ago
Would Love to hear about code refactor/optimization if any ...
George Al Ismail
@maotao11All comments
- @maotao11Posted over 3 years ago
Good solution friend! I did the same challenge recently and i have some tips that would made your solution better, first of all if you add the weights near the import of the font you will achieve the same font as the original preview example(@import url("https://fonts.google.com/specimen/Kumbh+Sans:200,400,700"); and if you nest the "80k", "803K", "1.3K" in <p> you can center them better to match the bottom text. Overall though great job!
0 - @WandoniumSubmitted over 3 years ago
Hey, guys. I think I used too many media queries on this solution. What do ya'll think?
@maotao11Posted over 3 years agoGood solution , but I think the height is kinda broken, at least on my end i see the letters leave the box, im not sure what causes it but probably its something related to the "body" tag or a flexbox issue.
0
