Skip to content

Conversation

@IlianaB
Copy link
Contributor

@IlianaB IlianaB commented Nov 19, 2025

related to #7729

@ui5-webcomponents-bot
Copy link
Collaborator

ui5-webcomponents-bot commented Nov 19, 2025

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview November 19, 2025 15:02 Inactive
@IlianaB IlianaB changed the title wip(ui5-li-custom): improve accessibility announcements feat(ui5-li-custom): improve accessibility announcements Dec 8, 2025
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview December 9, 2025 09:33 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview December 16, 2025 10:11 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview December 17, 2025 14:42 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview January 6, 2026 11:34 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview January 6, 2026 12:52 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview January 6, 2026 15:01 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview January 6, 2026 15:09 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview January 6, 2026 15:16 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview January 7, 2026 08:08 Inactive
@aborjinik
Copy link
Contributor

why don't we reuse this? We could externalize it from the table.
https://github.com/UI5/webcomponents/blob/main/packages/main/src/TableCustomAnnouncement.ts#L55

@IlianaB
Copy link
Contributor Author

IlianaB commented Jan 20, 2026

why don't we reuse this? We could externalize it from the table. https://github.com/UI5/webcomponents/blob/main/packages/main/src/TableCustomAnnouncement.ts#L55

We discussed with the team: we could use an externalized logic for extracting accessibility descriptions. The only concern - currently there is a specific table-related logic. Can it be separated from the externalized method for processing the accessibility info?

@aborjinik
Copy link
Contributor

Actually, I do not see any table-specific logic here. But if such logic exists, it should go into the options (we should probably use the second parameter of getAccessibilityDescription for options anyway).

  1. data-ui5-table-acc-text (we could generalize this to data-ui5-acc-text) can be used for native elements (or web components) to provide (or override) accessibility information.
  2. If there is nothing inside the custom list item, I think it would be acceptable to announce it as "Empty".
  3. If we cannot retrieve the accessibility information of an element and the default accessibility logic does not return any text, but the element is tabbable (for example, a simple native <input> element), we should not announce it as empty. Instead we announce something like "contains control", so the user knows that F2 can be used to jump into it for more details.

As mentioned I do not see those as table-specific.

@IlianaB
Copy link
Contributor Author

IlianaB commented Jan 22, 2026

Actually, I do not see any table-specific logic here. But if such logic exists, it should go into the options (we should probably use the second parameter of getAccessibilityDescription for options anyway).

  1. data-ui5-table-acc-text (we could generalize this to data-ui5-acc-text) can be used for native elements (or web components) to provide (or override) accessibility information.
  2. If there is nothing inside the custom list item, I think it would be acceptable to announce it as "Empty".
  3. If we cannot retrieve the accessibility information of an element and the default accessibility logic does not return any text, but the element is tabbable (for example, a simple native <input> element), we should not announce it as empty. Instead we announce something like "contains control", so the user knows that F2 can be used to jump into it for more details.

As mentioned I do not see those as table-specific.

Thanks for explanations @aborjinik . Let us know when it’s ready for use. We’ll try to adapt. If any questions or difficulties arise, we’ll contact you. :)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants