
Design comparison
Community feedback
- Account deleted
Pretty good. I reviewed the code, and I’d like to offer a few recommendations. For the card size, I suggest using width: 90% and max-width: 384px, for example, this way, the card will maintain the proper size without needing to rely on media queries.
Also, if your goal is to center the card, instead of using margin: 5/2rem, you could apply min-height: 100vh; display: grid; place-items: center; to the element that contains the card. Alternatively, you can use min-height: 100vh; display: flex; on that container and margin: auto; on the proper card.
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