Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

3-column preview card component

@YosefHayim

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Adriano 36,650

@AdrianoEscarabote

Posted

Hi Yosef Hayim, how are you doing? I really loved the outcome of your project, but I have a few suggestions that I think might be helpful:

I noticed that you use more than one h1, this is not a good practice since it is recommended to have only one h1 per page to inform the main title of the page! so remove all h1 and put h2

I noticed that you used a button in which case the best option would be an a, because in my head when a person clicks on a button written Learn More, he is not confirming a form, or something like, it will be redirected to another page, to Learn More about!

to solve this problem do this:

<a href="/">Learn More</a>

The rest is excellent.

I hope you find it useful. 👍

0
P
Lo-Deck 2,460

@Lo-Deck

Posted

Hi well done for this challenge,

For this <div class="first-container"> you can use <section class="first-container">

Use a div when there is no other choices, you can use section, article...

It's better to use em or rem instead of px. FreeCodeCamp. FreeCodeCamp.

Hope to be helpful.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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