Skip to content

Feature/multiple fonticons#262

Open
pvidoczi wants to merge 11 commits intomainfrom
feature/multiple-fonticons
Open

Feature/multiple fonticons#262
pvidoczi wants to merge 11 commits intomainfrom
feature/multiple-fonticons

Conversation

@pvidoczi
Copy link
Copy Markdown
Collaborator

@pvidoczi pvidoczi commented Feb 9, 2026

No description provided.

@pvidoczi pvidoczi requested a review from mercury83 as a code owner February 9, 2026 13:32
sizeCollection="small"
variant="light"
aria-hidden="true"
[fontSet]="'idsbase'"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Binding is not necessary here.


return Object.entries<DemoControlItem<T[keyof T]>>(controls)
.filter(([
, ctrl,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Comma accidentally?

<span class="icon">
@if (menuItem.icon) {
<ids-icon [fontIcon]="menuItem.icon" />
<ids-icon [fontSet]="'idsbase'" [fontIcon]="menuItem.icon" />
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Binding is not necessary for fontSet.

<span class="name">{{ menuItem.name! | translate }}</span>
</a>
<span #trigger class="more"><ids-icon [fontIcon]="menuItem.$open ? 'chevron-up' : 'chevron-down'" /></span>
<span #trigger class="more"><ids-icon [fontSet]="'idsbase'" [fontIcon]="menuItem.$open ? 'chevron-up' : 'chevron-down'" /></span>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Here too...

(click)="toggleDark()"
>
<ids-icon icon-leading [fontIcon]="isDark() ? 'light-mode' : 'dark-mode'" />
<ids-icon icon-leading [fontSet]="'idsbase'" [fontIcon]="isDark() ? 'light-mode' : 'dark-mode'" />
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

And here...

>
<ids-segmented-control-toggle-item value="light">
<ids-icon fontIcon="light-mode" />
<ids-icon fontIcon="light-mode" [fontSet]="'idsbase'" />
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Binding is not necessary (and below).

template: `
@if (dividerType() === _dividerType.ICON) {
<ids-icon aria-hidden="true" fontIcon="chevron-right" [size]="size()" />
<ids-icon aria-hidden="true" fontIcon="chevron-right" [fontSet]="'idsbase'" [size]="size()" />
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Binding is not necessary here.

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