@correlucas
Posted
πΎHello again Raz, congratulations for your new solution!
I was looking your previous solution and I've some tips for you:
First of all you can improve a bit the html semantics replacing the card divs with article
. Remember to use div only for small blocks of content and for the bigger blocks choose meaningful tags.
You've used <br>
to break the paragraph line but this is not a best practice because it gives your solution a strange behavior when the card grow in the mobile version and the text didnt grow to fit the card, you can control the paragraph size using max-width: 400px
to make the text break lines or even use a limit of characters like max-width: 40ch
to have each line having maximum 40 chracthres.
If you want to see my solution for the same challenge, here's the link: https://www.frontendmentor.io/solutions/3-column-preview-card-vanilla-css-custom-design-and-hover-states-e5uAfmUT71
π I hope this helps you and happy coding!
Marked as helpful
@PishoTo
Posted
@correlucas thanks again, Lucas!! So much information and precious feedback - priceless! Will have a look at your solution and in case I have any questions I will let you know if that's ok :) Thank you!