Submitted over 3 years ago
3-Col Preview Card Component using Flexbox
@rizwanmustafa
Design comparison
SolutionDesign
Solution retrospective
Any feedback is welcome!
Community feedback
- @ApplePieGiraffePosted over 3 years ago
Hi there, Rizwan Mustafa! 👋
Nice job on this challenge! 👏
A few things I'd like to suggest are,
- Switching to a mobile-friendly layout a little sooner to prevent a horizontal scroll bar from appearing along the bottom of the screen.
- Decreasing the space between the paragraph and the "Learn More" elements in the mobile layout.
- Only using a single
<h1>
on the page, since it is often recommended that there only be one of those elements per page (in this case, the headings of each section of the card component could simply be<h2>
s or<h3>
s).
Keep coding (and happy coding, too)! 😁
1@rizwanmustafaPosted over 3 years ago@ApplePieGiraffe Thanks, will surely follow these recommendations next time
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