Design comparison
Solution retrospective
Brushing up my basics, feel free to drop in advices to make the code better.
Community feedback
- @grace-snowPosted over 3 years ago
Hi
I’m afraid text is unreadable it’s so small on this (viewing on phone). The whole component is very small actually, and I can’t see the buttons at all. Once you’ve increased all the sizes and fixed mobile buttons, make sure you convert font sizes to REM, never use px for that
As well as the style issues, the learn more buttons should be using anchor tag elements, as they would trigger navigation to another page in a real site
Marked as helpful1@TSPraneethPosted over 3 years ago@grace-snow Thanks for the feedback, I'll look into it and make the required changes :)
0 - @RealMohamedAymanPosted over 3 years ago
Add "border-radius" It will help making the container looks better
Marked as helpful0@TSPraneethPosted over 3 years ago@RealMohamedAyman Thanks for the feedback, I'll look into it and make the required changes :)
1
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