Qr code component using CSS Flexbox

Community feedback

• 

  HashimAlSadah•860

  @HashimAlSadahPosted over 2 years ago

  Nice work completing this challenge. You should at least have one <main></main> container in your html code. (probably, you should replace <section> with <main>). Also you should replace h2 with h1 since you have to start by order h1, h2, h3, etc... have the attribution in <footer></footer>. (I have done all the mistakes above before :) ) These steps should solve most of your accessibility issues. Overall, great work.

  Marked as helpful

  2

  ---

  

  Luca Lagos•50

  @luca-lagosPosted over 2 years ago

  @HashimAlSadah Hi Hashim, thanks very much for your advices :).

  0
• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  Luca!! Felicitaciones por completar este reto!!

  Te recomendaría que implementes los cambios de que te mencionaron más arriba ya que eliminará todos los errores de accesibilidad que sale en el reporte, obviamente apenas subas los cambios actualiza el reporte para que salga en 0.

  Yo te iba a mencionar que el color del titulo de la card es color: rgb(31, 50, 81), respecto a lo demás te ha quedado bastante bien, revisaré tus demás trabajos a ver que puntos puedes mejorar.

  0