Skip to content

fix(ui5-list): fix acc finding#13558

Open
PetyaMarkovaBogdanova wants to merge 4 commits into
mainfrom
fix-acc-list
Open

fix(ui5-list): fix acc finding#13558
PetyaMarkovaBogdanova wants to merge 4 commits into
mainfrom
fix-acc-list

Conversation

@PetyaMarkovaBogdanova
Copy link
Copy Markdown
Contributor

@PetyaMarkovaBogdanova PetyaMarkovaBogdanova commented May 20, 2026

1) Tree (1 failing)

  • Failing area: noDataText API
  • Reported issue: list semantics mismatch (list item without proper list parent context)
  • Typical signal: continuum list-structure violation in the no-data row.
Tree noDataText row breaks list semantics list-structure violation around no-data row No-data container rendered with listitem-like semantics but not inside proper list/listbox/tree parent Likely real High Ensure no-data markup uses role-consistent container hierarchy; avoid orphan listitem semantics No node with role listitem/option/treeitem is outside expected parent role

Tree extends UI5Element

Exposes the noDataText property/API
Delegates rendering to TreeList component
TreeList extends List

Only overrides item flattening logic (getItems())
Reuses List's entire template and rendering
List component

Has the actual template that renders the no-data row
TreeList inherits this template without modification

@ui5-webcomponents-bot
Copy link
Copy Markdown
Collaborator

ui5-webcomponents-bot commented May 20, 2026

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 20, 2026 07:16 Inactive
@PetyaMarkovaBogdanova PetyaMarkovaBogdanova marked this pull request as ready for review May 20, 2026 14:35
@PetyaMarkovaBogdanova PetyaMarkovaBogdanova changed the title Fix acc list fix(ui5-list): fix acc finding May 20, 2026
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview May 20, 2026 14:43 Inactive
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.

2 participants