HTML, CSS unable to make it responsive. Sorry :-)
Design comparison
Solution retrospective
Please give feedback, my code is not up to the mark. The card isn't responsive, will go through it again. But it was fun and very productive IMO.
Community feedback
- @kirmintPosted over 3 years ago
Hello, adding to the comment above would suggest to use
em
sizes instead ofpx
. To add rounded corners you should useborder-radius
property: https://developer.mozilla.org/en-US/docs/Web/CSS/border-radius1@Samarth-5109Posted over 3 years ago@kirmint Thanks for the feedback. Yup! I will replace px with em and make it responsive also as I need more practice. So, will definitely do it.
P.S. -- Oh! I forgot to apply border-radius. Thanks for pointing it out.
0 - @vanzasetiaPosted over 3 years ago
Hi my name is Vanza, I have some feedback for you, in order to make it responsive you can add media query, using grid or flexbox and by the way to make the card in the center of the page you can use flexbox. display: flex; justify-content: center; align-items: center; and min-height: 100vh; You can add those properties on the body style and wrap all your card to one single div. You can see my code on my GitHub repo: https://github.com/vanzasetia/stats-preview-card-component .Hope this will help you to make it responsive!
1@Samarth-5109Posted over 3 years ago@vanzasetia Thanks, for the feedback. I will add a media query also to make it responsive as I am still struggling with grid and flexbox, need more practice. Will take a clue from your code too.
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord