article-preview

Community feedback

• 

  Abdul•8,540

  @SamadeenPosted over 2 years ago

  Hey!! Cheers 🥂 on completing this challenge.. .

  Here are my suggestions..

  • You should nest all your div and heading in a main tag**.
  • You can wrapper your attribution section in a footer tag to avoid accessibility issues.

  This should fix most of your accessibility issues

  . Regardless you did amazing... hope you find this useful... Happy coding!!!

  0

  ---

  

  Denis•1,060

  @Mod8124Posted over 2 years ago

  @Samadeen Cheers, I will keep that in mind, thank you.

  0
• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  Denis, buenas!

  Lo errores que tienes de Landmark se pueden solucionar rápidamente colocando todo lo que hay dentro del body además dentro de una etiqueta main, ya que siempre debes poner la información principal de tu proyecto dentro de esta etiqueta. Puedes dejar el div con la clase attribution fuera del main y ponerlo con la etiqueta footer ya depende de ti.

  Eso si intenta no usar las etiquetas h1, h2, h3... h6 sin ningún sentido semantico. Se utilizan para agregar titulos y subtitulos. Si o si necesitas una etiqueta h1 por página o te saldrá error, y luego las demás deben ir colocalas de manera incremental, es como cuando hacer un informe y este tiene un solo titulo y luego subtitulos por apartados y así sucesivamente, masomenos esa es la idea como deberías manejarlo.

  Por ahora solo he revisado la accesibilidad ya luego le doy una revisada al diseño y funcionalidad pero con eso debería salirte 0 errores luego de corregirlo.

  0

  ---

  

  Denis•1,060

  @Mod8124Posted over 2 years ago

  @Sdann26 Hola, buenas! Gracias por la respuesta lo corregire y lo tendre en cuenta para futuros projectos.

  1

  

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  @Mod8124 De nada Denis, éxitos :D!

  0