Design comparison
Solution retrospective
Don't have any questions but all feedback is very much appreciated
Community feedback
- @correlucasPosted over 2 years ago
Hello Yahir, congratulations for your solution!
The card component is perfect in the desktop version, but in the mobile one, the second column doesn't grow enought to keep all the text content. This is due some fixed
height
you've set in thedesktop
version.Look what isn't allowing the column grow in mobile:
.container { max-height: 430px;}
Note that you don't need to set the height, the container automatically grow with the element paddings and margins.
To keep this challenge simple my advice is that you used
display: grid;
to build the component with two equal column, its much easier.Hope it helps, and happy coding!
Marked as helpful0@Le-YzzzPosted over 2 years ago@correlucas thanks again 😅 looks like i have to learn grid next hehe
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