From 3603624b70b80bcf5ed154d7054e1f8a848e085f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ullrich=20Sch=C3=A4fer?= <hi+github@ullrich.is>
Date: Thu, 16 Jul 2026 00:43:30 +0200
Subject: [PATCH] fix(planner): distinct aria-label for export dropdown toggle
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The restyled Export split-button's dropdown toggle got aria-label
"Export GPX", colliding with the main button's text and breaking the
E2E selector that clicked the old "▾" glyph. Give it a dedicated
exportOptions label (en + de) and target it in the E2E test.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 apps/planner/app/components/ExportButton.tsx | 2 +-
 e2e/planner-routing.test.ts                  | 2 +-
 packages/i18n/src/locales/de.ts              | 1 +
 packages/i18n/src/locales/en.ts              | 1 +
 4 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/apps/planner/app/components/ExportButton.tsx b/apps/planner/app/components/ExportButton.tsx
index 34b786c..13fbe08 100644
--- a/apps/planner/app/components/ExportButton.tsx
+++ b/apps/planner/app/components/ExportButton.tsx
@@ -68,7 +68,7 @@ export function ExportButton({ yjs }: { yjs: YjsState }) {
         </button>
         <button
           onClick={(e) => { e.stopPropagation(); setOpen((v) => !v); }}
-          aria-label={t("exportGpx")}
+          aria-label={t("exportOptions")}
           aria-expanded={open}
           className={`${split} rounded-r-md px-2 text-text-md`}
         >
diff --git a/e2e/planner-routing.test.ts b/e2e/planner-routing.test.ts
index e1d38af..2ea7b5a 100644
--- a/e2e/planner-routing.test.ts
+++ b/e2e/planner-routing.test.ts
@@ -91,7 +91,7 @@ test.describe("Planner – routing", () => {
     const sidebar = page.locator("aside");
     await expect(sidebar.getByText("Refill water here")).toBeVisible({ timeout: 5000 });
 
-    await page.getByRole("button", { name: "▾" }).click();
+    await page.getByRole("button", { name: "Export options" }).click();
     await expect(page.getByText("Export Plan")).toBeVisible({ timeout: 3000 });
     const downloadPromise = page.waitForEvent("download");
     await page.getByText("Export Plan").first().click();
diff --git a/packages/i18n/src/locales/de.ts b/packages/i18n/src/locales/de.ts
index 3dcd4fc..a73bde6 100644
--- a/packages/i18n/src/locales/de.ts
+++ b/packages/i18n/src/locales/de.ts
@@ -18,6 +18,7 @@ export default {
     newSession: "Neue Sitzung",
     saveRoute: "Route speichern",
     exportGpx: "GPX exportieren",
+    exportOptions: "Export-Optionen",
     exportRoute: "Route exportieren",
     exportRouteDesc: "GPX-Track für jede App",
     exportPlan: "Plan exportieren",
diff --git a/packages/i18n/src/locales/en.ts b/packages/i18n/src/locales/en.ts
index 83493f35..3e93f72 100644
--- a/packages/i18n/src/locales/en.ts
+++ b/packages/i18n/src/locales/en.ts
@@ -18,6 +18,7 @@ export default {
     newSession: "New Session",
     saveRoute: "Save Route",
     exportGpx: "Export GPX",
+    exportOptions: "Export options",
     exportRoute: "Export Route",
     exportRouteDesc: "Clean GPX track for any app",
     exportPlan: "Export Plan",