Design comparison
SolutionDesign
Community feedback
- @grace-snowPosted almost 3 years ago
Really nice solution here, well done.
Only problem is these lines are problematic
width: 320px; height: 497px;
That width is causing the card to hit screen edges on my phone. To fix it, change it to max-width (Or max-inline-size)
The height should be removed altogether. You could use min-height if you really want, but in general you can let the content guide how big a height the browser paints
Marked as helpful0@grace-snowPosted almost 3 years agoOther tips
- try to use rem more instead of px, including for margins
- add classes to what you want to style directly rather than nesting selectors where possible to keep your css specificity low
- use margin to create vertical space between low level elements like headings and paragraphs (that's not what padding is for)
Marked as helpful0@steveWhoCodesPosted almost 3 years ago@grace-snow
Thank you so much for the feedback! I truly appreciate any advice given to help me become a better version of myself. And I learned a new trick, max-inline-size! Going to have to look into that one.
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