Submitted over 2 years ago

3 Column preview card with FlexBox

Dawid7600• 100

@Dawid7600

Preview site View code

This is a solution for...

3-column preview card component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

This is my first project with the Frontend Mentor website. I will be grateful for checking my code to be able to improve my skills and knowledge :)

Community feedback

Fidel Lim• 2,775

@fidellim

Posted over 2 years ago

Hi Dawid,

congrats on finishing your first FEM project! Well done! It looks great for mobile and desktop views. Just a suggestion I would like to share is that you can add hover and active states to your buttons. This will help users checking your site that those elements are clickable.

I hope it helps :)

Marked as helpful

Marlon Passos• 940

@MarlonPassos-git

Posted over 2 years ago

congratulations for your first challenge, some suggestions: Add this code in the body to center the card min-height: 100vh; display: grid; place-intens: center; .

Each of the sections could be inside a <section> tag and all the content could be inside a <main> tag, this would make the code more semantic and organized.
Seeing that you styled css using standard HTML tags, I would recommend you always use classes to style as this helps to avoid errors in the future

Marked as helpful

Please log in to post a comment

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