Skip to content

chore(badge): Resolve slotted icon presence#2091

Merged
rkaraivanov merged 1 commit intomasterfrom
rkaraivanov/badge-slotted-icon
Feb 9, 2026
Merged

chore(badge): Resolve slotted icon presence#2091
rkaraivanov merged 1 commit intomasterfrom
rkaraivanov/badge-slotted-icon

Conversation

@rkaraivanov
Copy link
Member

Description

Resolve slotted icon presence using the slot controller

Type of Change

  • Refactoring (code improvements without functional changes)

Checklist

  • My code follows the project's coding standards
  • I have tested my changes locally

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Refactors igc-badge to detect the presence of a slotted igc-icon via the shared slot controller, rather than using DOM querying.

Changes:

  • Register a slot controller instance on the component and route slot changes through it.
  • Update icon-presence detection to use hasAssignedElements(...) for the default slot.

@rkaraivanov rkaraivanov merged commit c2b6272 into master Feb 9, 2026
7 checks passed
@rkaraivanov rkaraivanov deleted the rkaraivanov/badge-slotted-icon branch February 9, 2026 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant