fix: resolve strict mode violation for duplicate dialog buttons in visual editor

renemadsen · claude · renemadsen · commit 8f744eca9bdb · 2026-04-05T08:35:39.000+02:00
When creating multiple nested checklists, Angular Material keeps previous
dialog elements in the DOM, causing #changeChecklistSaveCancelBtn and
#changeChecklistSaveBtn to match 2 elements. Use .last() to target the
topmost (most recent) dialog.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/eform-client/playwright/e2e/Page objects/EformVisualEditor.page.ts b/eform-client/playwright/e2e/Page objects/EformVisualEditor.page.ts
@@ -101,11 +101,11 @@ export class EformVisualEditorPage extends PageWithNavbarPage {
   }
 
   changeChecklistSaveCancelBtn(): Locator {
-    return this.page.locator('#changeChecklistSaveCancelBtn');
+    return this.page.locator('#changeChecklistSaveCancelBtn').last();
   }
 
   changeChecklistSaveBtn(): Locator {
-    return this.page.locator('#changeChecklistSaveBtn');
+    return this.page.locator('#changeChecklistSaveBtn').last();
   }
 
   async selectedLanguages(): Promise<number[]> {

Original file line number	Diff line number	Diff line change
`@@ -101,11 +101,11 @@ export class EformVisualEditorPage extends PageWithNavbarPage {`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`changeChecklistSaveCancelBtn(): Locator {`
`104`		`- return this.page.locator('#changeChecklistSaveCancelBtn');`
	`104`	`+ return this.page.locator('#changeChecklistSaveCancelBtn').last();`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`changeChecklistSaveBtn(): Locator {`
`108`		`- return this.page.locator('#changeChecklistSaveBtn');`
	`108`	`+ return this.page.locator('#changeChecklistSaveBtn').last();`
`109`	`109`	`}`
`110`	`110`
`111`	`111`	`async selectedLanguages(): Promise<number[]> {`