Bing SEO Analyzer complains about multiple <h1> tags in a single page

[Dan Scott]

* Evergreen master

Bing SEO Analyzer reports as a "high severity" issue:

"""
There are multiple <h1> tags on the page.

Recommended Action:
Remove redundant <h1> tags from the page source, so that only one <h1> tag exists.
"""

It looks like the screenreader-only effort that added <h1>Catalog</h1> to the record details page has resulted in this conflicting with the <h1> that we use for the title of the record itself--and the latter should be the <h1> for the page.

Isn't having an <h1>Catalog</h1> for screen readers redundant given the <title> that we have for the page? And given that a screen reader is going to give hierarchical access to the page, don't we want the <h1> to be the title of the record itself so that they can jump there?

[Shula Link (slink-g)]

Confirmed.

For the fix, the <h1>Catalog</h1> is in the Open-ILS/src/templates/opac/parts/base.tt2 file. However, removing that <h1> will mean all other catalog pages will have no <h1> attribute, which is also an SEO violation.

Maybe the fix would be a hierarchy like:

<h1>Catalog</h1>
<h2>Item Title</h2>
<h3>Everything Else</h3>

Would solve the problem for that page, at least.