article-preview

Community feedback

• 

  Abdul•8,560

  @Samadeen

  Posted about 3 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!!!

  01
• 

  Danilo Blas•6,300

  @Sdann26

  Posted about 3 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.

  02