Submitted almost 3 years ago

Html, CSS

Jeremy Mendoza• 100

@JeremyMen

Preview site View code

This is a solution for...

3-column preview card component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

Do you think that my style.css file is well organized? Thanks for your feedbacks

Community feedback

Himanshu Tegyalwar• 195

@himanshuat

Posted almost 3 years ago

This comment was deleted over 1 year ago

Jeremy Mendoza• 100

@JeremyMen

Posted almost 3 years ago

Hi @himanshuat, Thanks a lot for your feedback!

I have just a question: why "learn more" buttons should be anchor tags?

Thanks again,

Cheers

Rahul Baranwal• 135

@Rahulbaran

Posted almost 3 years ago

Hey Jeremy , I know you tried to make it similar as shown in the picture. But you need to work on certain things:-

line-height
padding in button
margin between text and button
use box-sizing:border-box in html(do google for this and also look for box-model)

Also you should learn about rem and em units & use them in padding, font-size etc.

I know there is a lot to research but you already know , doing research will make you a better developer.

Jeremy Mendoza• 100

@JeremyMen

Posted almost 3 years ago

Hey @Rahulbaran, Thank you for helping me!

You really enlightened me by telling me to use box-sizing and rem/em units! I didn't know them and I'm sure they'll come in handy.

Have a nice day

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