Skip to content

fix(jump-links): landmark label#2870

Merged
bennypowers merged 5 commits intomainfrom
fix/jump-links/landmark-label
Apr 15, 2026
Merged

fix(jump-links): landmark label#2870
bennypowers merged 5 commits intomainfrom
fix/jump-links/landmark-label

Conversation

@bennypowers
Copy link
Copy Markdown
Member

Closes #2860

What I did

  1. add aria-labelledby="label" to the nav landmark element

Testing Instructions

  1. confirm that screen readers read the page correctly

Notes to Reviewers

  1. Note that the listbox is already labelled by the same element.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Oct 27, 2024

🦋 Changeset detected

Latest commit: a5b23c9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@patternfly/elements Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Oct 27, 2024

✅ Commitlint tests passed!

More Info
{
  "valid": true,
  "errors": [],
  "warnings": [],
  "input": "fix(jump-links): landmark label"
}

@netlify
Copy link
Copy Markdown

netlify bot commented Oct 27, 2024

Deploy Preview for patternfly-elements ready!

Name Link
🔨 Latest commit 8e793d7
😎 Deploy Preview https://deploy-preview-2870--patternfly-elements.netlify.app/

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions github-actions bot added the AT passed Automated testing has passed label Oct 27, 2024
@github-actions
Copy link
Copy Markdown
Contributor

SSR Test Run for af81208: Report

@adamjohnson adamjohnson self-requested a review October 28, 2024 20:27
Copy link
Copy Markdown
Collaborator

@adamjohnson adamjohnson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noticed you assigned this to Greg and I. Was that intentional or did you want us to be reviewers?

Happy to take this up if applicable.

Comment thread elements/pf-jump-links/pf-jump-links.ts
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 5, 2024

SSR Test Run for 9c8154f: Report

@github-actions

This comment has been minimized.

@bennypowers bennypowers enabled auto-merge (squash) November 5, 2024 19:45
Copy link
Copy Markdown
Collaborator

@adamjohnson adamjohnson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Link Groups, Titled Meaningfully

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 15, 2026

Deploy Preview for patternfly-elements ready!

Name Link
🔨 Latest commit a5b23c9
🔍 Latest deploy log https://app.netlify.com/projects/patternfly-elements/deploys/69dfcc228f93bf0008bf796f
😎 Deploy Preview https://deploy-preview-2870--patternfly-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@bennypowers bennypowers merged commit 6b3031b into main Apr 15, 2026
15 checks passed
@bennypowers bennypowers deleted the fix/jump-links/landmark-label branch April 15, 2026 17:36
@github-actions
Copy link
Copy Markdown
Contributor

SSR Test Run for 8e793d7: Report

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AT passed Automated testing has passed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pf-jump-links shadow root nav container axeDevTools error ('Landmarks should have a unique role or role/label/title (i.e. accessible name) combination')

3 participants