News-homepage-main

Community feedback

• 

  Gustavo Gutiérrez•360

  @gustavo2023

  Posted about 1 month ago

  There are a couple of issues I noticed in your project:

  • First, it seems you are not using the correct font-family for the project.
  • There is an error when you close the sidebar because the goBack() function is not defined.
  • Instead of using a <div> element with a .nav class you should instead use a more semantic element like <header>. This helps improve the structure and navigation for assistive technologies.
  • You shouldn't use more than one <h1> heading on the same page. The <h1> tag is meant to represent the main heading or the most important title of the page. Having more than one <h1> can make the document's structure confusing for assistive technologies like screen readers, which rely on heading levels to navigate the content.
  • You also shouldn't skip heading levels. Skipping heading levels (e.g., going directly from <h1> to <h3> without an <h2>) can make it hard for screen readers to convey the content hierarchy to visually impaired users. They might not understand the relationship between sections or subsections.

  00