From be984dc05143bdc0a0ca571d9834a051217f37ec Mon Sep 17 00:00:00 2001
From: Ly Tempel <ly.tempel@bitweb.ee>
Date: Fri, 15 May 2026 20:20:50 +0300
Subject: [PATCH] fix(collapse): add button type to prevent form submission
 #434

---
 tedi/components/buttons/collapse/collapse.component.html | 1 +
 .../buttons/collapse/collapse.component.spec.ts          | 9 +++++++++
 2 files changed, 10 insertions(+)

diff --git a/tedi/components/buttons/collapse/collapse.component.html b/tedi/components/buttons/collapse/collapse.component.html
index fe310cba4..0d824c618 100644
--- a/tedi/components/buttons/collapse/collapse.component.html
+++ b/tedi/components/buttons/collapse/collapse.component.html
@@ -1,5 +1,6 @@
 <div class="tedi-collapse" [class.tedi-collapse--open]="isOpen() === true">
   <button
+    type="button"
     class="tedi-collapse__button"
     (click)="toggleCollapse()"
     [attr.aria-expanded]="isOpen() ? true : false"
diff --git a/tedi/components/buttons/collapse/collapse.component.spec.ts b/tedi/components/buttons/collapse/collapse.component.spec.ts
index f74882227..8364ade84 100644
--- a/tedi/components/buttons/collapse/collapse.component.spec.ts
+++ b/tedi/components/buttons/collapse/collapse.component.spec.ts
@@ -108,4 +108,13 @@ describe("CollapseComponent", () => {
     );
     expect(iconWrapper).toBeNull();
   });
+
+  it("should render the toggle button with type='button' to avoid submitting parent forms", () => {
+    const button: HTMLButtonElement = fixture.nativeElement.querySelector(
+      ".tedi-collapse__button",
+    );
+    expect(button).toBeTruthy();
+    expect(button.getAttribute("type")).toBe("button");
+    expect(button.type).toBe("button");
+  });
 });