Submitted about 1 year ago
Desktop-first 3-column preview card component
@quielLovesLasagna
Design comparison
SolutionDesign
Solution retrospective
Hi, tell me what you think
Community feedback
- @Zy8712Posted about 1 year ago
Your site and code look pretty good. Great job!!!
Some things I noticed when reading through your code and testing your site:
- when the browser window is small, a part of your Sedan column gets cut off and users cannot read it even when using the horizontal scroll bar. To fix this you can set your the css foy your body to
align-items: flex-start
when the window size is small. - you are missing mobile support. You can add in mobile support by changing the css for your items inside a media query. Examples of media query use here.
- you should add in a
<main>
tag to wrap around your content. This is mainly for accessibility purposes - it is better to use
<button>
over<a>
for the "Learn More" buttons
Hope you find this feedback useful 👍
0@quielLovesLasagnaPosted about 1 year ago@Zy8712, thanks for your feedback! I purposely did not include the media query in this solution XD. Will take your feedback into account.
0 - when the browser window is small, a part of your Sedan column gets cut off and users cannot read it even when using the horizontal scroll bar. To fix this you can set your the css foy your body 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