html css flex-box

Community feedback

• 

  Adriano•38,100

  @AdrianoEscarabotePosted about 2 years ago

  Hi Luis, how are you?

  I really liked the result of your project, but I have some tips that I think you will enjoy:

  As you know this project is based only on one page component, so no h1 is needed! because we don't know if this page will have a more important component, but it's always good to prevent accessibility errors so I think it would be good for you to add an h1 in this component, besides being a good practice for when you are developing larger sites, don't worry forget about h1.

  to solve this change h2 to h1.

  I noticed that to make the bold effect, you used a p tag inside the h2, prefer to use a span!

  The rest is great!

  I hope it helps... 👍

  0