3-column-preview-card-component using HTML and CSS
Design comparison
Solution retrospective
I finished this under 4hours. Getting better day by day!π
Any suggestions on how I can improve are welcome! Thank you!
Community feedback
- @elaineleungPosted about 2 years ago
Hi Rotimi, great work, and glad to hear that you can see yourself improving every day π
I think you did many things well here, including using
sr-only
,aria-hidden
for icons, and also usinga
for the call to action button, and so there's not a lot to comment on. The only comment I have is just for the call to action button in mobile mode, as it looks like its width got smaller and everything looks squished to the point where the text is no readable. This could be due to thewidth: 3.438rem
in mobile mode. I think you might not need that, so try removing it and see. Also, you can try usingarticle
instead ofdiv
for each of the cards, as they are like individual posts and can act as standalone items.Well done, and keep going! π
Marked as helpful1@TimiphilPosted about 2 years ago@elaineleung Thank you so much Elaine. I really appreciate your comment. I remove the width and it works well.π π
0 - @correlucasPosted about 2 years ago
πΎHello Rotimi , Congratulations on completing this challenge!
Youβve done really good work here putting everything together, Iβve some suggestions you can consider applying to your code to improve the html markup/semantic:
Improve your html markup using meaningful tags to wrap the content, you can replace the div youβve used for each card with <article>. Remember to wrap big blocks of content with semantic tags and never divs, use divs for small blocks.
Your solution seems fine, you did a really good job wrapping the content for these 3 cards. Something you can improve here is to use a
single class
to manage the content that is mostly the same for the 3 cards (paddings, colors, margins and etc) and another class to manage the characteristics that are different (colors and icon), this way you'll have more control over then and if you need to change something you modify only one class.βοΈ I hope this helps you and happy coding!
Marked as helpful0@TimiphilPosted about 2 years ago@correlucas Thank you for this comment. it is really helpful and it meant a lot to me. πππ
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