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.

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

[BUG]: A11Y: About page should not have multiple h1 elements #167

Open
dgodongm opened this issue Oct 2, 2023 · 1 comment
Open

[BUG]: A11Y: About page should not have multiple h1 elements #167

dgodongm opened this issue Oct 2, 2023 · 1 comment
Assignees
Labels
bug Something isn't working needs triage Determine priority of issues

Comments

@dgodongm
Copy link
Contributor

dgodongm commented Oct 2, 2023

About page has multiple h1 elements.

Expected Behavior

Web pages should only have one h1 and it should generally be for the title

Current Behavior

About page has 2 h1 elements

Possible Solution

At a high level components should not default to having h1. Should be able to have heading level set dynamically based on context. If a default level is specified, then it should be h2.

Steps to Reproduce

  1. Navigate to About page (http://localhost:3000/about when running locally)
  2. Inspect page or run HeadingsMap browser add-on

Screenshot:
Screenshot 2023-10-02 at 4 50 10 PM

@dgodongm dgodongm added bug Something isn't working needs triage Determine priority of issues labels Oct 2, 2023
@kelvinsekx
Copy link

I would love to pick this up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage Determine priority of issues
Projects
None yet
Development

No branches or pull requests

3 participants