Column Preview Card Component Main / Grid and Flex

Solution retrospective
I'm happy with the result. If you have any suggestions or advice please share :)
Please log in to post a comment
Log in with GitHubCommunity feedback
- P@Islandstone89
Hi, good job on this challenge. Just a few things to take note of:
HTML:
- "Learn More" would navigate to another page, hence it should be a link, not a button.
CSS:
-
On
main
,height:100vh
should bemin-height: 100vh
. -
It's good practice to include a CSS Reset at the top.
-
Font-size must never be in px. Use rem, or em like you have done elsewhere.
Marked as helpful - @FluffyKas
Hey,
This looks great! Just a few small things you could perhaps do:
- remove alt text from icons: as they are decorative, they can be left blank
- you could add some left and right margin to the container - on certain screen sizes (770-920px) it's touching the sides of the screen
The rest looks awesome, well done!
Marked as helpful
Join 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