From f305a47c7065d1f60e3f1fb5010ec3052c7dd1af Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?David=20Louren=C3=A7o?= <david.lourenco@outsystems.com>
Date: Tue, 5 May 2026 18:57:44 +0100
Subject: [PATCH] fix(segment): segment drag would set disabled segment button
 checked

---
 .../segment-view/test/basic/index.html        | 16 +++++++++
 .../test/basic/segment-view.e2e.ts            | 33 +++++++++++++++++++
 core/src/components/segment/segment.tsx       |  4 +++
 3 files changed, 53 insertions(+)

diff --git a/core/src/components/segment-view/test/basic/index.html b/core/src/components/segment-view/test/basic/index.html
index 78dec1d9ff9..631a6a69a57 100644
--- a/core/src/components/segment-view/test/basic/index.html
+++ b/core/src/components/segment-view/test/basic/index.html
@@ -79,6 +79,22 @@
           <ion-segment-content id="top">Top</ion-segment-content>
         </ion-segment-view>
 
+        <ion-segment id="disabledFirstSegment" value="third">
+          <ion-segment-button value="first" disabled>
+            <ion-label>First</ion-label>
+          </ion-segment-button>
+          <ion-segment-button content-id="second-content" value="second">
+            <ion-label>Second</ion-label>
+          </ion-segment-button>
+          <ion-segment-button content-id="third-content" value="third">
+            <ion-label>Third</ion-label>
+          </ion-segment-button>
+        </ion-segment>
+        <ion-segment-view id="disabledFirstSegmentView">
+          <ion-segment-content id="second-content">Second Content</ion-segment-content>
+          <ion-segment-content id="third-content">Third Content</ion-segment-content>
+        </ion-segment-view>
+
         <ion-segment value="peach" scrollable>
           <ion-segment-button content-id="orange" value="orange">
             <ion-label>Orange</ion-label>
diff --git a/core/src/components/segment-view/test/basic/segment-view.e2e.ts b/core/src/components/segment-view/test/basic/segment-view.e2e.ts
index f3f35626eaa..4bd434d18f2 100644
--- a/core/src/components/segment-view/test/basic/segment-view.e2e.ts
+++ b/core/src/components/segment-view/test/basic/segment-view.e2e.ts
@@ -169,5 +169,38 @@ configs({ modes: ['md'] }).forEach(({ title, config }) => {
       const segmentContent = page.locator('ion-segment-content[id="top"]');
       await expect(segmentContent).toBeInViewport();
     });
+    test('should not select a disabled first segment button when dragging segment-view from last content', async ({
+      page,
+    }) => {
+      await page.setContent(
+        `
+        <ion-segment id="disabledFirstSegment" scrollable="true" value="third">
+          <ion-segment-button value="first" disabled>
+            <ion-label>First</ion-label>
+          </ion-segment-button>
+          <ion-segment-button content-id="second-content" value="second">
+            <ion-label>Second</ion-label>
+          </ion-segment-button>
+          <ion-segment-button content-id="third-content" value="third">
+            <ion-label>Third</ion-label>
+          </ion-segment-button>
+        </ion-segment>
+        <ion-segment-view id="disabledFirstSegmentView">
+          <ion-segment-content id="second-content">Second Content</ion-segment-content>
+          <ion-segment-content id="third-content">Third Content</ion-segment-content>
+        </ion-segment-view>
+      `,
+        config
+      );
+      await page.waitForChanges();
+
+      await page.locator('ion-segment-view').evaluate((el: HTMLElement) => {
+        const max = el.scrollWidth - el.clientWidth;
+        el.scrollLeft = max;
+      });
+      await page.waitForChanges();
+
+      await expect(page.locator('ion-segment-button[value="first"]')).not.toHaveClass(/segment-button-checked/);
+    });
   });
 });
diff --git a/core/src/components/segment/segment.tsx b/core/src/components/segment/segment.tsx
index 6a56f8fdddf..e680f22413b 100644
--- a/core/src/components/segment/segment.tsx
+++ b/core/src/components/segment/segment.tsx
@@ -400,6 +400,10 @@ export class Segment implements ComponentInterface {
 
       const nextIndex = Math.round(scrollRatio * (buttons.length - 1));
 
+      if (buttons[nextIndex]?.disabled) {
+        return;
+      }
+
       if (this.lastNextIndex === undefined || this.lastNextIndex !== nextIndex) {
         this.lastNextIndex = nextIndex;
         this.triggerScrollOnValueChange = false;