Skip to content

feat(distribution): allow custom icon component for inquiry status - #3083

Merged
anehx merged 1 commit into
projectcaluma:mainfrom
anehx:distribution-custom-icon
Aug 12, 2026
Merged

feat(distribution): allow custom icon component for inquiry status#3083
anehx merged 1 commit into
projectcaluma:mainfrom
anehx:distribution-custom-icon

Conversation

@anehx

@anehx anehx commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Status configs can now define an iconComponent which is rendered instead of the default UIkit icon. The icon rendering of the inquiry divider and the navigation status indicator is extracted into the new CdInquiryStatusIcon component, which either renders the configured component or falls back to the previous UkIcon with tooltip.

Color class and tooltip are only applied to the default icon - a custom component receives @status and @tooltipPos, and is responsible for rendering color and tooltip itself.

@anehx
anehx requested a review from StephanH90 August 12, 2026 08:17
Status configs can now define an `iconComponent` which is rendered
instead of the default `UIkit` icon. The icon rendering of the inquiry
divider and the navigation status indicator is extracted into the new
`CdInquiryStatusIcon` component, which either renders the configured
component or falls back to the previous `UkIcon` with tooltip.

Color class and tooltip are only applied to the default icon - a custom
component receives `@status` and `@tooltipPos`, and is responsible for
rendering color and tooltip itself.
@anehx
anehx force-pushed the distribution-custom-icon branch from 8b29974 to 05b3405 Compare August 12, 2026 08:20

@StephanH90 StephanH90 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

AY LGTM.

@anehx
anehx merged commit 396d57b into projectcaluma:main Aug 12, 2026
65 checks passed
@anehx
anehx deleted the distribution-custom-icon branch August 12, 2026 08:28
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