Bing SEO Analyzer complains about multiple <h1> tags in a single page
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Confirmed
|
Undecided
|
Unassigned |
Bug Description
* 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?
tags: | added: accessibility opac |
Changed in evergreen: | |
status: | New → Confirmed |
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.