From 1d3ba956cbd6dbd8c25024c74e55931911bd8354 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:05:23 +0200
Subject: [PATCH 1/2] feat(planner): restyle Export + SaveToJournal buttons on
 tokens

Bring the two slotted topbar actions onto the design system so the bar
reads as one cohesive surface:

- SaveToJournalButton: primary Button primitive; "saved" uses the
  accent, error uses a new --color-danger token, and the return link
  gets secondary token styling.
- ExportButton: token split-button (secondary look, chevron icon,
  single divider) and a token dropdown menu (raised surface, soft
  shadow, muted descriptions).
- New --color-danger token (#a03c3c, the no-go hue at full strength)
  for error text.

Behavior and i18n unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 apps/planner/app/components/ExportButton.tsx  | 44 +++++++++----------
 .../app/components/SaveToJournalButton.tsx    | 18 ++++----
 packages/ui/src/theme.css                     |  1 +
 3 files changed, 32 insertions(+), 31 deletions(-)

diff --git a/apps/planner/app/components/ExportButton.tsx b/apps/planner/app/components/ExportButton.tsx
index 87ceabc..34b786c 100644
--- a/apps/planner/app/components/ExportButton.tsx
+++ b/apps/planner/app/components/ExportButton.tsx
@@ -52,45 +52,45 @@ export function ExportButton({ yjs }: { yjs: YjsState }) {
 
   const hasMultipleDays = hasDayBreaks(yjs);
 
+  const item =
+    "block w-full px-3 py-1.5 text-left transition-colors hover:bg-bg-subtle";
+  const split =
+    "h-7 border border-border bg-bg-raised text-sm font-medium text-text-hi transition-colors hover:bg-bg-subtle focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-accent";
+
   return (
     <div ref={ref} className="relative z-[1001]">
       <div className="flex">
         <button
           onClick={handleExportRoute}
-          className="rounded-l bg-gray-100 px-3 py-1 text-sm text-gray-700 hover:bg-gray-200"
+          className={`${split} rounded-l-md border-r-0 px-3`}
         >
           {t("exportGpx")}
         </button>
         <button
           onClick={(e) => { e.stopPropagation(); setOpen((v) => !v); }}
-          className="rounded-r border-l border-gray-300 bg-gray-100 px-2.5 py-1 text-sm text-gray-700 hover:bg-gray-200"
+          aria-label={t("exportGpx")}
+          aria-expanded={open}
+          className={`${split} rounded-r-md px-2 text-text-md`}
         >
-          ▾
+          <svg width="12" height="12" viewBox="0 0 24 24" fill="none" stroke="currentColor" strokeWidth="2.5" strokeLinecap="round" strokeLinejoin="round" aria-hidden>
+            <polyline points="6 9 12 15 18 9" />
+          </svg>
         </button>
       </div>
       {open && (
-        <div className="absolute left-0 top-full z-50 mt-1 w-56 rounded border border-gray-200 bg-white py-1 shadow-lg">
-          <button
-            onClick={handleExportRoute}
-            className="block w-full px-3 py-1.5 text-left hover:bg-gray-100"
-          >
-            <span className="text-sm text-gray-700">{t("exportRoute")}</span>
-            <span className="block text-xs text-gray-400">{t("exportRouteDesc")}</span>
+        <div className="absolute left-0 top-full z-50 mt-1 w-56 rounded-md border border-border bg-bg-raised py-1 shadow-md">
+          <button onClick={handleExportRoute} className={item}>
+            <span className="text-sm text-text-hi">{t("exportRoute")}</span>
+            <span className="block text-xs text-text-lo">{t("exportRouteDesc")}</span>
           </button>
-          <button
-            onClick={handleExportPlan}
-            className="block w-full px-3 py-1.5 text-left hover:bg-gray-100"
-          >
-            <span className="text-sm text-gray-700">{t("exportPlan")}</span>
-            <span className="block text-xs text-gray-400">{t("exportPlanDesc")}</span>
+          <button onClick={handleExportPlan} className={item}>
+            <span className="text-sm text-text-hi">{t("exportPlan")}</span>
+            <span className="block text-xs text-text-lo">{t("exportPlanDesc")}</span>
           </button>
           {hasMultipleDays && (
-            <button
-              onClick={handleExportDays}
-              className="block w-full px-3 py-1.5 text-left hover:bg-gray-100"
-            >
-              <span className="text-sm text-gray-700">{t("exportDays")}</span>
-              <span className="block text-xs text-gray-400">{t("exportDaysDesc")}</span>
+            <button onClick={handleExportDays} className={item}>
+              <span className="text-sm text-text-hi">{t("exportDays")}</span>
+              <span className="block text-xs text-text-lo">{t("exportDaysDesc")}</span>
             </button>
           )}
         </div>
diff --git a/apps/planner/app/components/SaveToJournalButton.tsx b/apps/planner/app/components/SaveToJournalButton.tsx
index d464cf4..217d6e7 100644
--- a/apps/planner/app/components/SaveToJournalButton.tsx
+++ b/apps/planner/app/components/SaveToJournalButton.tsx
@@ -1,5 +1,6 @@
 import { useState, useCallback } from "react";
 import { useTranslation } from "react-i18next";
+import { Button } from "@trails-cool/ui";
 import { computeSurfaceBreakdown } from "@trails-cool/map-core";
 import type { YjsState } from "~/lib/use-yjs";
 import { buildPlanGpx } from "~/lib/gpx-export";
@@ -64,17 +65,16 @@ export function SaveToJournalButton({ yjs, sessionId, returnUrl }: SaveToJournal
 
   return (
     <div className="flex items-center gap-2">
-      <button
-        onClick={handleSave}
-        disabled={saving}
-        className="rounded bg-green-600 px-3 py-1 text-sm text-white hover:bg-green-700 disabled:opacity-50"
-      >
+      <Button variant="primary" size="sm" onClick={handleSave} disabled={saving}>
         {saving ? t("saving") : t("saveToJournal")}
-      </button>
-      {saved && <span className="text-xs text-green-600">{t("saved")}</span>}
-      {error && <span className="text-xs text-red-600">{error}</span>}
+      </Button>
+      {saved && <span className="text-xs text-accent">{t("saved")}</span>}
+      {error && <span className="text-xs text-danger">{error}</span>}
       {saved && returnUrl && (
-        <a href={returnUrl} className="rounded bg-gray-100 px-2 py-1 text-xs text-gray-700 hover:bg-gray-200">
+        <a
+          href={returnUrl}
+          className="inline-flex h-7 items-center rounded-md border border-border bg-bg-raised px-2.5 text-xs font-medium text-text-hi transition-colors hover:bg-bg-subtle"
+        >
           {t("returnToJournal")}
         </a>
       )}
diff --git a/packages/ui/src/theme.css b/packages/ui/src/theme.css
index b3135d4..8bb178d 100644
--- a/packages/ui/src/theme.css
+++ b/packages/ui/src/theme.css
@@ -41,6 +41,7 @@
   --color-stop-border: rgba(139, 109, 58, 0.25);
 
   /* ── Danger / no-go ── */
+  --color-danger: #a03c3c; /* solid danger — error text, matches the no-go hue */
   --color-nogo: rgba(160, 60, 60, 0.12);
   --color-nogo-border: rgba(160, 60, 60, 0.3);
 

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 2/2] 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",