From ba814fd6ec262f2bedc7b97044af4347421d3076 Mon Sep 17 00:00:00 2001 From: AleksanderSklorz <115619721+AleksanderSklorz@users.noreply.github.com> Date: Fri, 24 Jul 2026 13:38:40 +0200 Subject: [PATCH] [ACS-8467] Removed duplicated dialog after clicking label in folder rules creation dialog (#12088) --- .../card-view-textitem.component.html | 2 +- .../card-view-textitem.component.spec.ts | 24 +++++++++++++++++++ .../card-view-textitem.component.ts | 3 ++- 3 files changed, 27 insertions(+), 2 deletions(-) diff --git a/lib/core/src/lib/card-view/components/card-view-textitem/card-view-textitem.component.html b/lib/core/src/lib/card-view/components/card-view-textitem/card-view-textitem.component.html index c771772e0a..79d3fe7460 100644 --- a/lib/core/src/lib/card-view/components/card-view-textitem/card-view-textitem.component.html +++ b/lib/core/src/lib/card-view/components/card-view-textitem/card-view-textitem.component.html @@ -107,7 +107,7 @@ class="adf-textitem-clickable" [ngClass]="{ 'adf-property-read-only': isReadonlyProperty }" [attr.data-automation-id]="'card-textitem-toggle-' + property.key" - (click)="clicked()" + (click)="clicked($event)" > { expect(callBackSpy).toHaveBeenCalled(); }); + it('should call preventDefault on event when label of clickable input is clicked', () => { + component.property.clickable = true; + component.ngOnChanges({}); + fixture.detectChanges(); + + const clickEvent = new MouseEvent('click', { bubbles: true }); + spyOn(clickEvent, 'preventDefault'); + testingUtils.getByDataAutomationId(`card-textitem-label-${component.property.key}`).nativeElement.dispatchEvent(clickEvent); + fixture.detectChanges(); + expect(clickEvent.preventDefault).toHaveBeenCalled(); + }); + + it('should call preventDefault on event when button wrapper of clickable input is clicked', () => { + component.property.clickable = true; + component.ngOnChanges({}); + fixture.detectChanges(); + + const clickEvent = new MouseEvent('click'); + spyOn(clickEvent, 'preventDefault'); + testingUtils.getByDataAutomationId(`card-textitem-toggle-${component.property.key}`).nativeElement.dispatchEvent(clickEvent); + fixture.detectChanges(); + expect(clickEvent.preventDefault).toHaveBeenCalled(); + }); + it('should click event to the event stream when clickable property enabled', async () => { const cardViewUpdateService = TestBed.inject(CardViewUpdateService); spyOn(cardViewUpdateService, 'clicked').and.stub(); diff --git a/lib/core/src/lib/card-view/components/card-view-textitem/card-view-textitem.component.ts b/lib/core/src/lib/card-view/components/card-view-textitem/card-view-textitem.component.ts index 02cf2fee7d..a944f91d18 100644 --- a/lib/core/src/lib/card-view/components/card-view-textitem/card-view-textitem.component.ts +++ b/lib/core/src/lib/card-view/components/card-view-textitem/card-view-textitem.component.ts @@ -203,12 +203,13 @@ export class CardViewTextItemComponent extends BaseCardView