ampproject / ampproject/amp.dev

Accessibility: "Follow us" header tag in footer breaks Lighthouse audits.

Open
#4,431 0 comments 0 reactions 0 assignees View on GitHub
Category: Accessibility Type: Bug
Dominant language
HTML
Stars
600
Forks
675
PR merge metrics
No merged PRs in 30d

Description

## 🐛 Bug Report
"Follow us" header tag in site-level footer breaks Lighthouse audits.
![image](https://user-images.githubusercontent.com/10456171/92026493-1d5b7400-ed2f-11ea-986b-4f18b0b5c105.png)

### Current behavior
Failing lighthouse accessibility audit: `Heading elements are not in a sequentially-descending order`
[Relevant resource](http://web.dev/heading-order/?utm_source=lighthouse)

### Expected behavior/code
Appropriate heading hierarchy identifying the section. We can fix this by replacing `

` with ``, making sure styles also transfer as needed.

### Reproduction procedure
1. Go to amp.dev
2. Open Lighthouse in Chrome DevTools
3. Run an Accessibility audit
4. See findings

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.