Skip to content

fix(ui5-list): suppress F2 aria description when no interactive items present#13489

Merged
NakataCode merged 4 commits into
mainfrom
ui5-list-f2-aria-description
Jun 12, 2026
Merged

fix(ui5-list): suppress F2 aria description when no interactive items present#13489
NakataCode merged 4 commits into
mainfrom
ui5-list-f2-aria-description

Conversation

@NakataCode

@NakataCode NakataCode commented May 8, 2026

Copy link
Copy Markdown
Contributor

Problem

When the List has selectionMode="None" and contains only plain standard items, the screen reader incorrectly announces "To move to the content, press F2" via aria-description on the <ul> element.

This affects the Tokenizer in read-only mode, which sets selectionMode="None" on its internal List in the n-more popover, causing the misleading instruction to be announced even though there are no interactive elements to reach with F2.


Solution

Introduce a _hasInteractiveItems getter that gates the F2 instruction, returning true only when any of the following conditions are met:

  • selectionMode is Delete — the custom delete button slot may be tabbable
  • Any item has type="Detail" — the detail button is tabbable
  • Any item is a ListItemCustom — arbitrary tabbable slot content

Important Note:

  • The F2 aria-description is now only announced when the list actually contains interactive elements that can be reached with F2. Previously it was announced on every list regardless of content.

Fixes: #13347

@NakataCode NakataCode requested a review from dobrinyonkov May 8, 2026 13:13
@ui5-webcomponents-bot

ui5-webcomponents-bot commented May 8, 2026

Copy link
Copy Markdown
Collaborator

🧹 Preview deployment cleaned up: https://pr-13489--ui5-webcomponents.netlify.app

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 8, 2026 13:15 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 8, 2026 15:36 Inactive
Comment thread packages/main/src/List.ts Outdated
Comment thread packages/main/cypress/specs/List.cy.tsx
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 19, 2026 11:25 Inactive
@NakataCode NakataCode temporarily deployed to netlify-preview June 3, 2026 07:33 — with GitHub Actions Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview June 3, 2026 07:40 Inactive
@NakataCode NakataCode merged commit f27c1ff into main Jun 12, 2026
60 of 69 checks passed
@NakataCode NakataCode deleted the ui5-list-f2-aria-description branch June 12, 2026 07:33
@NakataCode NakataCode temporarily deployed to netlify-preview June 12, 2026 07:33 — with GitHub Actions Inactive
@sap-ui5-webcomponents-release

Copy link
Copy Markdown

🎉 This PR is included in version v2.24.0-rc.1 🎉

The release is available on v2.24.0-rc.1

Your semantic-release bot 📦🚀

@sap-ui5-webcomponents-release

Copy link
Copy Markdown

🎉 This PR is included in version v2.24.0 🎉

The release is available on v2.24.0

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Tokenizer]: Incorrect F2 instruction announced in read-only mode for tokenizer

3 participants