[go: nahoru, domu]
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Heading Tags For HTML Report #7575

Closed
jhomme opened this issue Mar 19, 2019 · 2 comments
Closed

Heading Tags For HTML Report #7575

jhomme opened this issue Mar 19, 2019 · 2 comments
Labels
needs-priority report

Comments

@jhomme
Copy link
jhomme commented Mar 19, 2019

Hi,
To make it easier for screen reader users to view the HTML report, would it be possible to make the bold section headers use heading mark-up, as in <h?> tags? I found this when using Lighthouse from the command line as a Node module. If this is user error on my part, I apologize. I'm just learning JavaScript and am dangerous to myself right now. :)
@johnemau
Copy link
Contributor

+1
Making the report gauges headings would allow screen reader users to easily navigate between the headings directly.

Here is a screenshot of an empty heading list provided by the NVDA screen reader Element List feature which would otherwise be used to jump between headings.

gauge-should-be-h2

@johnemau johnemau mentioned this issue Jul 20, 2019
Merged
@brendankenny
Copy link
Member

User names are very close here, I thought at first you were +1ing yourself :)

Closed in #9422

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs-priority report
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@johnemau @brendankenny @patrickhulce @connorjclark @jhomme