From 1fba2ff1818e8ce899a85d4ad02042c9d7ba4960 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=87etin?= <69278826+cetincakiroglu@users.noreply.github.com> Date: Tue, 12 Dec 2023 18:58:33 +0300 Subject: [PATCH] Fixed #14295 --- src/app/components/menu/menu.ts | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/src/app/components/menu/menu.ts b/src/app/components/menu/menu.ts index 11617e6b987..d6b14c02659 100755 --- a/src/app/components/menu/menu.ts +++ b/src/app/components/menu/menu.ts @@ -22,6 +22,7 @@ import { ViewEncapsulation, ViewRef, computed, + effect, forwardRef, signal } from '@angular/core'; @@ -117,8 +118,6 @@ export class MenuItemContent { @Input() itemTemplate: HTMLElement | undefined; - @Input() id: string; - @Output() onMenuItemClick: EventEmitter = new EventEmitter(); menu: Menu; @@ -128,7 +127,7 @@ export class MenuItemContent { } onItemClick(event, item) { - this.onMenuItemClick.emit({ originalEvent: event, item: { ...item, id: this.id } }); + this.onMenuItemClick.emit({ originalEvent: event, item }); } } /** @@ -194,7 +193,7 @@ export class MenuItemContent { [ngClass]="{ 'p-hidden': item.visible === false || submenu.visible === false, 'p-focus': focusedOptionId() && menuitemId(item, id, i, j) === focusedOptionId(), 'p-disabled': disabled(item.disabled) }" [ngStyle]="item.style" [class]="item.styleClass" - (onMenuItemClick)="itemClick($event)" + (onMenuItemClick)="itemClick($event, menuitemId(item, id, i, j))" pTooltip [tooltipOptions]="item.tooltipOptions" role="menuitem" @@ -217,7 +216,7 @@ export class MenuItemContent { [ngClass]="{ 'p-hidden': item.visible === false, 'p-focus': focusedOptionId() && menuitemId(item, id, i, j) === focusedOptionId(), 'p-disabled': disabled(item.disabled) }" [ngStyle]="item.style" [class]="item.styleClass" - (onMenuItemClick)="itemClick($event)" + (onMenuItemClick)="itemClick($event, menuitemId(item, id, i))" pTooltip [tooltipOptions]="item.tooltipOptions" role="menuitem" @@ -653,7 +652,7 @@ export class Menu implements OnDestroy { } } - itemClick(event: any) { + itemClick(event: any, id: string) { const { originalEvent, item } = event; if (item.disabled) { @@ -676,8 +675,8 @@ export class Menu implements OnDestroy { this.hide(); } - if (!this.popup && this.focusedOptionIndex() !== item.id) { - this.focusedOptionIndex.set(item.id); + if (!this.popup && this.focusedOptionIndex() !== id) { + this.focusedOptionIndex.set(id); } }