From 7348781b89178589036620b33f3554b2e7271c5f Mon Sep 17 00:00:00 2001
From: Ariel Costas Guerrero <ariel@costas.dev>
Date: Fri, 30 Jan 2026 19:29:55 +0100
Subject: feat: Enhance header and app shell with back navigation and dynamic
 title support

---
 src/frontend/app/routes/planner.tsx    |  13 ++--
 src/frontend/app/routes/routes-$id.tsx | 110 +++++++++++++++++++++------------
 2 files changed, 75 insertions(+), 48 deletions(-)

(limited to 'src/frontend/app/routes')

diff --git a/src/frontend/app/routes/planner.tsx b/src/frontend/app/routes/planner.tsx
index 32c37c0..1f64590 100644
--- a/src/frontend/app/routes/planner.tsx
+++ b/src/frontend/app/routes/planner.tsx
@@ -11,7 +11,7 @@ import LineIcon from "~/components/LineIcon";
 import { PlannerOverlay } from "~/components/PlannerOverlay";
 import { AppMap } from "~/components/shared/AppMap";
 import { APP_CONSTANTS } from "~/config/constants";
-import { usePageTitle } from "~/contexts/PageTitleContext";
+import { useBackButton, usePageTitle } from "~/contexts/PageTitleContext";
 import { type Itinerary } from "~/data/PlannerApi";
 import { usePlanner } from "~/hooks/usePlanner";
 import "../tailwind-full.css";
@@ -185,6 +185,7 @@ const ItineraryDetail = ({
   onClose: () => void;
 }) => {
   const { t } = useTranslation();
+  useBackButton({ onBack: onClose });
   const mapRef = useRef<MapRef>(null);
   const { destination: userDestination } = usePlanner();
   const [nextArrivals, setNextArrivals] = useState<
@@ -319,7 +320,8 @@ const ItineraryDetail = ({
               );
 
               if (resp.ok) {
-                arrivalsByStop[stopKey] = await resp.json() satisfies ConsolidatedCirculation[];
+                arrivalsByStop[stopKey] =
+                  (await resp.json()) satisfies ConsolidatedCirculation[];
               }
             } catch (err) {
               console.warn(
@@ -463,13 +465,6 @@ const ItineraryDetail = ({
             />
           </Source>
         </AppMap>
-
-        <button
-          onClick={onClose}
-          className="absolute top-4 left-4 bg-white dark:bg-slate-800 p-2 px-4 rounded-lg shadow-lg z-10 font-semibold text-sm hover:bg-slate-50 dark:hover:bg-slate-700 transition-colors"
-        >
-          {t("planner.back")}
-        </button>
       </div>
 
       {/* Details Panel */}
diff --git a/src/frontend/app/routes/routes-$id.tsx b/src/frontend/app/routes/routes-$id.tsx
index 3b61ba6..62de642 100644
--- a/src/frontend/app/routes/routes-$id.tsx
+++ b/src/frontend/app/routes/routes-$id.tsx
@@ -1,5 +1,5 @@
 import { useQuery } from "@tanstack/react-query";
-import { useRef, useState } from "react";
+import { useMemo, useRef, useState } from "react";
 import { useTranslation } from "react-i18next";
 import {
   AttributionControl,
@@ -10,7 +10,11 @@ import {
 import { useParams } from "react-router";
 import { fetchRouteDetails } from "~/api/transit";
 import { AppMap } from "~/components/shared/AppMap";
-import { usePageTitle } from "~/contexts/PageTitleContext";
+import {
+  useBackButton,
+  usePageTitle,
+  usePageTitleNode,
+} from "~/contexts/PageTitleContext";
 import "../tailwind-full.css";
 
 export default function RouteDetailsPage() {
@@ -35,6 +39,38 @@ export default function RouteDetailsPage() {
       : t("routes.details", "Detalles de ruta")
   );
 
+  const titleNode = useMemo(() => {
+    if (!route) {
+      return (
+        <span className="text-base font-semibold text-text">
+          {t("routes.details", "Detalles de ruta")}
+        </span>
+      );
+    }
+
+    return (
+      <div className="flex flex-col min-w-0">
+        <div className="flex items-center gap-2">
+          <span
+            className="text-lg font-bold leading-none"
+            style={{
+              color: route.color ? `#${route.color}` : "var(--text-color)",
+            }}
+          >
+            {route.shortName || route.longName}
+          </span>
+          <span className="text-sm text-text/90 truncate text-wrap tracking-tight leading-none">
+            {route.longName}
+          </span>
+        </div>
+      </div>
+    );
+  }, [route, t]);
+
+  usePageTitleNode(titleNode);
+
+  useBackButton({ to: "/routes" });
+
   if (isLoading) {
     return (
       <div className="flex justify-center py-12">
@@ -122,38 +158,6 @@ export default function RouteDetailsPage() {
 
   return (
     <div className="flex flex-col h-full overflow-hidden">
-      <div className="p-4 bg-surface border-b border-border">
-        <select
-          className="w-full p-2 rounded-lg border border-border bg-background text-text focus:ring-2 focus:ring-primary outline-none"
-          value={selectedPattern?.id}
-          onChange={(e) => {
-            setSelectedPatternId(e.target.value);
-            setSelectedStopId(null);
-          }}
-        >
-          {Object.entries(patternsByDirection).map(([dir, patterns]) => (
-            <optgroup
-              key={dir}
-              label={
-                dir === "0"
-                  ? t("routes.direction_outbound", "Ida")
-                  : t("routes.direction_inbound", "Vuelta")
-              }
-            >
-              {patterns.map((pattern) => (
-                <option key={pattern.id} value={pattern.id}>
-                  {pattern.code
-                    ? `${parseInt(pattern.code.slice(-2)).toString()}: `
-                    : ""}
-                  {pattern.headsign || pattern.name}{" "}
-                  {t("routes.trip_count_short", { count: pattern.tripCount })}
-                </option>
-              ))}
-            </optgroup>
-          ))}
-        </select>
-      </div>
-
       <div className="flex-1 flex flex-col overflow-hidden">
         <div className="flex-1 flex flex-col relative overflow-hidden">
           <div className="h-1/2 relative">
@@ -179,11 +183,7 @@ export default function RouteDetailsPage() {
               showTraffic={false}
               attributionControl={false}
             >
-              <AttributionControl
-                position="bottom-right"
-                compact={false}
-                customAttribution={route.agencyName || undefined}
-              />
+              <AttributionControl position="bottom-left" compact={true} />
               {selectedPattern?.geometry && (
                 <Source type="geojson" data={geojson}>
                   <Layer
@@ -214,6 +214,38 @@ export default function RouteDetailsPage() {
             </AppMap>
           </div>
 
+          <select
+            className="px-4 py-2 box-border bg-surface text-text focus:ring-2 focus:ring-primary outline-none"
+            value={selectedPattern?.id}
+            onChange={(e) => {
+              setSelectedPatternId(e.target.value);
+              setSelectedStopId(null);
+            }}
+          >
+            {Object.entries(patternsByDirection).map(([dir, patterns]) => (
+              <optgroup
+                key={dir}
+                label={
+                  dir === "0"
+                    ? t("routes.direction_outbound", "Ida")
+                    : t("routes.direction_inbound", "Vuelta")
+                }
+              >
+                {patterns.map((pattern) => (
+                  <option key={pattern.id} value={pattern.id}>
+                    {pattern.code
+                      ? `${parseInt(pattern.code.slice(-2)).toString()}: `
+                      : ""}
+                    {pattern.headsign || pattern.name}{" "}
+                    {t("routes.trip_count_short", {
+                      count: pattern.tripCount,
+                    })}
+                  </option>
+                ))}
+              </optgroup>
+            ))}
+          </select>
+
           <div className="flex-1 overflow-y-auto p-4 bg-background">
             <h3 className="text-lg font-bold mb-4">
               {t("routes.stops", "Paradas")}
-- 
cgit v1.3