Easybank landing page

Community feedback

• 

  Ahmed Kamel•1,190

  @hmadamkPosted about 2 years ago

  to solve your accissibility issues and Html issues you need to do the following

  1. Header does not need a role it's a header so that's already define it
  2. your section shouldn't have a role, role is specified for certain things to learn more about roles check mdn it should have aria-role for screen readers
  3. your page must have one section to be instead of a section to the main-element this should be the most important section like the hero one
  4. don't leave the srcset empty
  5. if the img is not for decorating the page you should but a description for it in the alt attribute
  6. your page must have one h1 element holding the main idea of your site you could wrap the img at the .header__logo with an h1 and then remove the margin from it what this does is that it assign the value of the alt to the h1 which mean your page is more search engine friendly
  7. you should use h2-h6 at least one into all of your sections happy coding hope this helped if it did help you consider marking this comment as helpful

  Marked as helpful

  1