Skip to content

Commit

Permalink
fix(core/menu): use label property to render internal menu items (#1293)
Browse files Browse the repository at this point in the history
  • Loading branch information
danielleroux authored May 23, 2024
1 parent 53f7744 commit fb48f05
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 12 deletions.
5 changes: 5 additions & 0 deletions .changeset/tame-shrimps-press.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@siemens/ix": patch
---

fix(core/menu): use label property to render internal menu items
20 changes: 8 additions & 12 deletions packages/core/src/components/menu/menu.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -681,9 +681,8 @@ export class Menu {
}}
icon={'cogwheel'}
onClick={async () => this.toggleSettings(!this.showSettings)}
>
{this.i18nSettings}
</ix-menu-item>
label={this.i18nSettings}
></ix-menu-item>
) : null}
<div onClick={(e) => this.onMenuItemsClick(e)}>
<slot name="bottom"></slot>
Expand All @@ -701,9 +700,8 @@ export class Menu {
}}
icon={'info'}
onClick={async () => this.toggleAbout(!this.showAbout)}
>
{this.i18nLegal}
</ix-menu-item>
label={this.i18nLegal}
></ix-menu-item>
) : null}
{this.enableToggleTheme ? (
<ix-menu-item
Expand All @@ -712,9 +710,8 @@ export class Menu {
onClick={() => themeSwitcher.toggleMode()}
class="internal-tab bottom-tab"
icon={'light-dark'}
>
{this.i18nToggleTheme}
</ix-menu-item>
label={this.i18nToggleTheme}
></ix-menu-item>
) : null}
{this.enableMapExpand || this.applicationLayoutContext?.sidebar ? (
<ix-menu-item
Expand All @@ -723,9 +720,8 @@ export class Menu {
onClick={() => this.sidebarToggle()}
class="internal-tab bottom-tab"
icon={`${this.getCollapseIcon()}`}
>
{this.getCollapseText()}
</ix-menu-item>
label={this.getCollapseText()}
></ix-menu-item>
) : null}
</aside>
<div
Expand Down

0 comments on commit fb48f05

Please sign in to comment.