Consider allowing header[role=banner]

[WilcoFiers]

  <main>
    <header role="banner">
      Hello world
    </header>
  </main>

This fails aria-allowed-role, in addition to failing landmark-banner-is-top-level. It's probably better for axe-core to only fail it under the top-level rule. It's a little odd to allow role=banner on a header if it's at the top, but to fail it if it's inside another landmark.

The same problem exists for other landmarks. I think the fix is to add banner to the header element in standards/html-elms.js.

[WilcoFiers]

Actually it'd be pretty cool if axe-core explicitly reported on the top-level rule to remove the role attribute.

[padmavemulapati]

Validated with the latest develop branch code base, now its showing only one rule failing for the code snippet

<main>
    <header role="banner">
      Hello world
    </header>
  </main>

when header is having role=banner

Previouly both aria-allowed-role and landmark rules failing.