
Design comparison
SolutionDesign
Community feedback
- P@YonerfyPosted 4 days ago
Hey T23harms, great job on this one! It looks really good.
There are a few things you need to fix to make it even better:
- Your text is overlapping because you set a fixed height in your card content. Try removing the height or setting it to
auto
. - You did well with the width by using percentages, but it's better to use relative units instead of pixels. This will help the layout adapt better to mobile and tablet screens.
- Adding a border to the image will make it match the design more closely.
- Don't forget to remove the template README.
Great job, mate! Keep going—you’re doing really well. 🙂
0@AJsemPosted 2 days ago@Yonerfy It is a static page, he should just made the page for the right container much or big enough for the content. no need to be scrolling in desktop view
0 - Your text is overlapping because you set a fixed height in your card content. Try removing the height or setting it to
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