Skip to content

Conversation

@pskelin
Copy link
Contributor

@pskelin pskelin commented Feb 9, 2026

Summary

  • Fix AvatarGroup to properly handle items passed through transitive slots
  • Add _slottedItems getter that uses getSlottedNodes to resolve nested slot elements
  • Replace all internal usages of this.items with this._slottedItems

When AvatarGroup is used inside another component that forwards content via a <slot> element, this.items returns the slot element itself instead of the actual avatar items. This fix ensures proper item resolution regardless of slot nesting depth.

Test plan

  • All existing AvatarGroup tests pass (20/20)
  • TypeScript compilation passes
  • Verify AvatarGroup works correctly when items come from a transitive slot

When items are passed through a transitive slot (e.g., when AvatarGroup
is used inside another component that forwards content via a slot),
this.items returns the slot element itself instead of the actual avatars.

Use getSlottedNodes to properly resolve transitive slotting and retrieve
the actual avatar items regardless of nesting depth.
@ui5-webcomponents-bot
Copy link
Collaborator

ui5-webcomponents-bot commented Feb 9, 2026

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

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview February 9, 2026 13:39 Inactive
@vladitasev vladitasev merged commit 1d17d50 into main Feb 10, 2026
21 of 22 checks passed
@vladitasev vladitasev deleted the avatar-group-slotted branch February 10, 2026 09:27
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview February 10, 2026 09:27 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.

4 participants