Standardise onboarding header sizes

Add Best Value small widget and standardise Closest wording
Clarify fuel savings comparison labels
2026-10-15 17:23:25 +01:00 · 2026-10-15 17:14:02 +01:00 · 2026-10-15 11:38:24 +01:00 · 2026-10-14 16:22:31 +01:00 · 2026-10-14 15:19:20 +01:00 · 2026-10-14 15:02:29 +01:00
22 changed files with 654 additions and 178 deletions
@@ -533,7 +533,7 @@
 				CLANG_WARN__DUPLICATE_METHOD_MATCH = YES;
 				CODE_SIGN_ENTITLEMENTS = Config/WatchApp.entitlements;
 				CODE_SIGN_STYLE = Automatic;
-				CURRENT_PROJECT_VERSION = 26;
+				CURRENT_PROJECT_VERSION = 27;
 				DEVELOPMENT_TEAM = F3BE6NE7U3;
 				ENABLE_PREVIEWS = YES;
 				ENABLE_USER_SCRIPT_SANDBOXING = YES;
@@ -599,7 +599,7 @@
 				CLANG_WARN__DUPLICATE_METHOD_MATCH = YES;
 				CODE_SIGN_ENTITLEMENTS = Config/WatchApp.entitlements;
 				CODE_SIGN_STYLE = Automatic;
-				CURRENT_PROJECT_VERSION = 26;
+				CURRENT_PROJECT_VERSION = 27;
 				DEVELOPMENT_TEAM = F3BE6NE7U3;
 				ENABLE_PREVIEWS = YES;
 				ENABLE_USER_SCRIPT_SANDBOXING = YES;
@@ -663,7 +663,7 @@
 				CLANG_WARN_UNGUARDED_AVAILABILITY = YES_AGGRESSIVE;
 				CLANG_WARN__DUPLICATE_METHOD_MATCH = YES;
 				CODE_SIGN_STYLE = Automatic;
-				CURRENT_PROJECT_VERSION = 26;
+				CURRENT_PROJECT_VERSION = 27;
 				DEVELOPMENT_TEAM = F3BE6NE7U3;
 				ENABLE_PREVIEWS = YES;
 				ENABLE_USER_SCRIPT_SANDBOXING = YES;
@@ -728,7 +728,7 @@
 				CLANG_WARN_UNGUARDED_AVAILABILITY = YES_AGGRESSIVE;
 				CLANG_WARN__DUPLICATE_METHOD_MATCH = YES;
 				CODE_SIGN_STYLE = Automatic;
-				CURRENT_PROJECT_VERSION = 26;
+				CURRENT_PROJECT_VERSION = 27;
 				DEVELOPMENT_TEAM = F3BE6NE7U3;
 				ENABLE_PREVIEWS = YES;
 				ENABLE_USER_SCRIPT_SANDBOXING = YES;
@@ -826,7 +826,7 @@
 				CODE_SIGN_ENTITLEMENTS = Config/App.entitlements;
 				CODE_SIGN_IDENTITY = "Apple Development";
 				CODE_SIGN_STYLE = Automatic;
-				CURRENT_PROJECT_VERSION = 26;
+				CURRENT_PROJECT_VERSION = 27;
 				DEVELOPMENT_TEAM = F3BE6NE7U3;
 				GENERATE_INFOPLIST_FILE = NO;
 				INFOPLIST_FILE = "Config/App-Info.plist";
@@ -851,7 +851,7 @@
 				CODE_SIGN_ENTITLEMENTS = Config/App.entitlements;
 				CODE_SIGN_IDENTITY = "Apple Development";
 				CODE_SIGN_STYLE = Automatic;
-				CURRENT_PROJECT_VERSION = 26;
+				CURRENT_PROJECT_VERSION = 27;
 				DEVELOPMENT_TEAM = F3BE6NE7U3;
 				GENERATE_INFOPLIST_FILE = NO;
 				INFOPLIST_FILE = "Config/App-Info.plist";
@@ -875,7 +875,7 @@
 				APPLICATION_EXTENSION_API_ONLY = YES;
 				CODE_SIGN_ENTITLEMENTS = Config/Widget.entitlements;
 				CODE_SIGN_STYLE = Automatic;
-				CURRENT_PROJECT_VERSION = 26;
+				CURRENT_PROJECT_VERSION = 27;
 				DEVELOPMENT_TEAM = F3BE6NE7U3;
 				GENERATE_INFOPLIST_FILE = NO;
 				INFOPLIST_FILE = "Config/Widget-Info.plist";
@@ -900,7 +900,7 @@
 				APPLICATION_EXTENSION_API_ONLY = YES;
 				CODE_SIGN_ENTITLEMENTS = Config/Widget.entitlements;
 				CODE_SIGN_STYLE = Automatic;
-				CURRENT_PROJECT_VERSION = 26;
+				CURRENT_PROJECT_VERSION = 27;
 				DEVELOPMENT_TEAM = F3BE6NE7U3;
 				GENERATE_INFOPLIST_FILE = NO;
 				INFOPLIST_FILE = "Config/Widget-Info.plist";
@@ -1523,7 +1523,6 @@ struct ContentView: View {
            fuelSavingsSettings: fuelSavingsSettings,
            fuelSavingsAmount: $sessionFuelSavingsAmount,
            savingsForStation: savingsLookup(baseline: baselineStation, journeys: scaffold.journeys),
            savingsBaselineStationName: baselineStation?.name,
            journeyDiagnosticsForStation: { station in journeyDiagnostics(for: station, baseline: baselineStation) },
            onToggleFavourite: toggleFavourite,
            onRefresh: { await refresh(force: true) }
@@ -1545,7 +1544,6 @@ struct ContentView: View {
            fuelSavingsSettings: fuelSavingsSettings,
            fuelSavingsAmount: $sessionFuelSavingsAmount,
            savingsForStation: savingsLookup(baseline: baselineStation, journeys: scaffold.journeys),
            savingsBaselineStationName: baselineStation?.name,
            journeyDiagnosticsForStation: { station in journeyDiagnostics(for: station, baseline: baselineStation) },
            onToggleFavourite: toggleFavourite,
            onReorder: reorderFavourites,
@@ -1944,7 +1942,6 @@ struct StationRow: View {
                FuelSavingsSummaryView(
                    availability: fuelSavings,
                    amount: fuelSavingsAmount,
                    baselineStationName: nil,
                    compact: true,
                    distanceUnit: distanceUnit
                )
@@ -21,10 +21,9 @@ struct FavouritesView: View {
    /// still never touches the Settings default.
    let fuelSavingsSettings: FuelSavingsSettings
    @Binding var fuelSavingsAmount: FuelSavingsFillAmount
-    /// Estimated saving for a favourite, against the nearest eligible station —
+    /// Estimated saving for a favourite, against the closest eligible station by
-    /// the same baseline and definition the Stations list uses.
+    /// journey distance — the same baseline and definition the Stations list uses.
    var savingsForStation: (FuelStation) -> FuelSavingsAvailability? = { _ in nil }
    var savingsBaselineStationName: String?
    var journeyDiagnosticsForStation: (FuelStation) -> FuelSavingsJourneyDiagnostics? = { _ in nil }
    var onToggleFavourite: (FuelStation, FuelType) -> Void = { _, _ in }
    /// Persists a reordered favourites array (after drag-and-drop).
@@ -84,7 +83,6 @@ struct FavouritesView: View {
         fuelSavingsSettings: FuelSavingsSettings = .defaults,
         fuelSavingsAmount: Binding<FuelSavingsFillAmount>,
         savingsForStation: @escaping (FuelStation) -> FuelSavingsAvailability? = { _ in nil },
         savingsBaselineStationName: String? = nil,
         journeyDiagnosticsForStation: @escaping (FuelStation) -> FuelSavingsJourneyDiagnostics? = { _ in nil },
         onToggleFavourite: @escaping (FuelStation, FuelType) -> Void = { _, _ in },
         onReorder: @escaping ([FavouriteEntry]) -> Void = { _ in },
@@ -99,7 +97,6 @@ struct FavouritesView: View {
        self.fuelSavingsSettings = fuelSavingsSettings
        _fuelSavingsAmount = fuelSavingsAmount
        self.savingsForStation = savingsForStation
        self.savingsBaselineStationName = savingsBaselineStationName
        self.journeyDiagnosticsForStation = journeyDiagnosticsForStation
        self.onToggleFavourite = onToggleFavourite
        self.onReorder = onReorder
@@ -152,7 +149,7 @@ struct FavouritesView: View {
                                referencePricePencePerLitre: cheapestPrice
                            )
                        } footer: {
-                            Text(String(localized: "Savings below are estimated against the nearest station with a price for this fuel — the same baseline as the Stations list.", comment: "Favourites fuel-savings explanation"))
+                            Text(String(localized: "Savings below are estimated against the closest eligible station by journey distance — the same baseline as the Stations list.", comment: "Favourites fuel-savings explanation"))
                        }
                    }
@@ -221,7 +218,6 @@ struct FavouritesView: View {
                    distanceUnit: distanceUnit,
                    fuelSavings: savingsForStation(station),
                    fuelSavingsAmount: fuelSavingsAmount,
                    savingsBaselineStationName: savingsBaselineStationName,
                    journeyDiagnostics: journeyDiagnosticsForStation(station),
                    // Every station listed here IS a favourite for the tab's
                    // fuel, so the star starts filled; tapping it unstars.
@@ -21,7 +21,7 @@ struct FuelBoardApp: App {
    var body: some Scene {
        WindowGroup {
-            ContentView()
+            rootView
                .onOpenURL(perform: handleOpenURL)
                .onReceive(NotificationCenter.default.publisher(for: .fuelBoardWatchRefreshRequested)) { _ in
                    WatchSyncManager.shared.pushSnapshot()
@@ -32,6 +32,24 @@ struct FuelBoardApp: App {
        }
    }
    /// QA hook (Debug builds only): `-qaScreenMarker` paints the whole screen a
    /// flat magenta. A macOS capture of the Simulator window can then locate the
    /// device-screen rectangle exactly, which yields a pixel-accurate mapping
    /// from simulator screenshots to click coordinates. That is the only way to
    /// drive REAL taps into the Simulator: `simctl` cannot, and this project has
    /// no UI-test target. Everything else renders ContentView as normal.
    @ViewBuilder private var rootView: some View {
        #if DEBUG
        if ProcessInfo.processInfo.arguments.contains("-qaScreenMarker") {
            Color(red: 1, green: 0, blue: 1).ignoresSafeArea()
        } else {
            ContentView()
        }
        #else
        ContentView()
        #endif
    }
    #if DEBUG
    private func startQALiveActivity(fuel: FuelType) {
        let state = FuelBoardLiveActivityAttributes.ContentState(
@@ -73,11 +73,16 @@ struct FuelSavingsAdjustmentView: View {
 struct FuelSavingsSummaryView: View {
    let availability: FuelSavingsAvailability?
    let amount: FuelSavingsFillAmount
-    let baselineStationName: String?
+    var baselineStationName: String? = nil
    var compact: Bool = false
    /// Sizes the "extra return travel" caption in the user's own unit (the
    /// stored figure is miles) — used by both the row banner and the detail sheet.
    var distanceUnit: DistanceUnit = .miles
    var showInfoAction: (() -> Void)? = nil
    private func comparisonLabel(for result: FuelSavingsResult) -> String {
        FuelSavingsFormat.comparisonLabel(status: result.status)
    }
    var body: some View {
        switch availability {
@@ -92,8 +97,16 @@ struct FuelSavingsSummaryView: View {
                // break even, blue baseline. lineLimit + minimumScaleFactor keep
                // it to one line, so rows stay even.
                VStack(alignment: .leading, spacing: 2) {
                    HStack(alignment: .firstTextBaseline, spacing: 6) {
                        capsule(for: result, compact: true)
                            .accessibilityLabel(accessibilityHeadline(for: result))
                        Text(comparisonLabel(for: result))
                            .font(.caption2)
                            .foregroundStyle(.secondary)
                            .lineLimit(1)
                            .layoutPriority(1)
                            .accessibilityLabel("Comparison: \(comparisonLabel(for: result))")
                    }
                    if let extraTravel = FuelSavingsFormat.extraTravelLabel(
                        extraDistanceMiles: result.extraDistanceMiles,
                        unit: distanceUnit,
@@ -117,8 +130,16 @@ struct FuelSavingsSummaryView: View {
                // room for. The old shape led with a status word + headline that
                // matched neither the listing nor each other.
                VStack(alignment: .leading, spacing: 10) {
                    HStack(alignment: .firstTextBaseline, spacing: 6) {
                        capsule(for: result, compact: false)
                            .accessibilityLabel(accessibilityHeadline(for: result))
                        Text(comparisonLabel(for: result))
                            .font(.subheadline)
                            .foregroundStyle(.secondary)
                            .lineLimit(1)
                            .layoutPriority(1)
                            .accessibilityLabel("Comparison: \(comparisonLabel(for: result))")
                    }
                    LabeledContent("Your fill-up", value: amountText)
                    LabeledContent("Fuel you'll receive", value: String(format: "%.2f litres", result.fillLitres))
                        .monospacedDigit()
@@ -133,11 +154,11 @@ struct FuelSavingsSummaryView: View {
                            .accessibilityLabel(spokenExtraTravel(for: result, fallback: extraTravel))
                    }
                    if result.status == .baseline {
-                        Text("Your closest station with a price for this fuel — every other saving is measured against it.")
+                        Text("Your closest eligible station by journey distance is the baseline — every other saving is measured against it.")
                            .font(.caption)
                            .foregroundStyle(.secondary)
                    } else {
-                        Text("Compared with \(baselineStationName ?? "your nearest station").")
+                        Text("Compared with \(baselineStationName ?? "your closest station").")
                            .font(.caption)
                            .foregroundStyle(.secondary)
                        LabeledContent(
@@ -166,6 +187,22 @@ struct FuelSavingsSummaryView: View {
                            .font(.caption2)
                            .foregroundStyle(.secondary)
                    }
                    if let showInfoAction {
                        Rectangle()
                            .fill(Color(.separator))
                            .frame(height: 0.5)
                            .padding(.vertical, 4)
                        Button(action: showInfoAction) {
                            HStack(spacing: 8) {
                                Text("How Best Value works")
                                Spacer(minLength: 8)
                                Image(systemName: "chevron.forward")
                                    .font(.footnote.weight(.semibold))
                                    .foregroundStyle(.tertiary)
                            }
                        }
                        .buttonStyle(.plain)
                    }
                }
            }
        case .unavailable(let reason):
@@ -217,7 +254,6 @@ struct FuelSavingsSummaryView: View {
                .accessibilityHidden(true)
            Text(result.capsuleHeadline())
                .lineLimit(1)
                .minimumScaleFactor(0.8)
        }
        .font(compact ? .caption.weight(.semibold) : .headline)
        .foregroundStyle(capsuleColor(for: result.status))
@@ -0,0 +1,69 @@
 import SwiftUI
 /// The "How Best Value works" explanation as its own sheet.
 ///
 /// It used to sit inline in the Key sheet as a `Section`; it is one shared view
 /// now so the Key sheet and the More info sheet can both open it, and the copy is
 /// unchanged (the same `Localizable.strings` keys are reused — the old section
 /// header string is the sheet's title).
 struct HowBestValueSheet: View {
    @Environment(\.dismiss) private var dismiss
    var body: some View {
        NavigationStack {
            List {
                Section {
                    VStack(alignment: .leading, spacing: 10) {
                        Text("Best Value compares each station with the closest eligible station by journey distance. It works out what you could save at the pump, then takes off the fuel used for the extra return journey.")
                        Text("Your fill-up amount and MPG are used to show whether you could save overall.")
                        Text("Results are marked “Approx.” until road distances are available.")
                            .foregroundStyle(.secondary)
                    }
                    .font(.subheadline)
                    .padding(.vertical, 4)
                }
            }
            .navigationTitle("How Best Value works")
            .navigationBarTitleDisplayMode(.inline)
            .toolbar {
                ToolbarItem(placement: .confirmationAction) {
                    Button("Done") { dismiss() }
                }
            }
        }
        .presentationDetents([.fraction(0.42)])
        .presentationBackground(Color(UIColor.systemGroupedBackground))
    }
 }
 /// The row that opens `HowBestValueSheet`. A tappable row with a chevron, like
 /// the other rows in both sheets. `.buttonStyle(.plain)` keeps it from becoming a
 /// row-wide tap target that could swallow a neighbouring control (see the
 /// two-buttons-in-one-row rule).
 struct HowBestValueRow: View {
    let action: () -> Void
    var body: some View {
        Button(action: action) {
            HStack(spacing: 8) {
                Text("How Best Value works")
                Spacer(minLength: 8)
                Image(systemName: "chevron.forward")
                    .font(.footnote.weight(.semibold))
                    .foregroundStyle(.tertiary)
            }
        }
        .buttonStyle(.plain)
    }
 }
 #if DEBUG
 /// QA hook support: `-showBestValueInfo` opens this sheet without a tap, so it can
 /// be screenshotted (simctl has no tap command) — the same pattern as
 /// `-showKeySheet` / `-showDetail`.
 enum BestValueInfoQA {
    static var isRequested: Bool {
        ProcessInfo.processInfo.arguments.contains("-showBestValueInfo")
    }
 }
 #endif
@@ -98,7 +98,7 @@ struct OnboardingView: View {
    private var welcomePage: some View {
        VStack(spacing: 0) {
            Spacer()
-            AnimatedOnboardingIcon(tint: .accentColor) {
+            AnimatedOnboardingIcon(tint: .accentColor, showsHalo: false) {
                // The app icon itself, with the standard iOS rounded-corner
                // treatment (~22% of the size), so onboarding matches the
                // Home Screen.
@@ -108,9 +108,9 @@ struct OnboardingView: View {
                    .frame(width: 96, height: 96)
                    .clipShape(RoundedRectangle(cornerRadius: 21, style: .continuous))
            }
-            .padding(.bottom, 28)
+            .padding(.bottom, 16)
-            Text(String(localized: "Welcome to FuelBoard", comment: "Onboarding welcome title"))
+            Text(String(localized: "Welcome to\nFuelBoard", comment: "Onboarding welcome title"))
                .font(.largeTitle.bold())
                .multilineTextAlignment(.center)
@@ -121,12 +121,13 @@ struct OnboardingView: View {
                .padding(.horizontal, 32)
                .padding(.top, 12)
-            featureRow(icon: "globe.europe.africa.fill", text: String(localized: "UK-wide prices — 8,000+ stations, updated twice a day", comment: "Onboarding feature row"))
+            featureRow(icon: "map.fill", title: String(localized: "Compare fuel prices near you", comment: "Onboarding feature row title"), subtitle: String(localized: "Over 8,000 stations across Great Britain, with prices updated daily.", comment: "Onboarding feature row subtitle"))
-            featureRow(icon: "scope", text: String(localized: "Cheapest within your chosen radius, or closest station first", comment: "Onboarding feature row"))
+            
-            featureRow(icon: "star.fill", text: String(localized: "Favourites with instant price comparison", comment: "Onboarding feature row"))
+            featureRow(icon: "chart.line.uptrend.xyaxis", title: String(localized: "See how much you could save", comment: "Onboarding feature row title"), subtitle: String(localized: "Compare fill-up costs and potential savings at different stations.", comment: "Onboarding feature row subtitle"))
-            featureRow(icon: "chart.bar.fill", text: String(localized: "Green/amber/red rating vs the best nearby price", comment: "Onboarding feature row"))
+            
-            featureRow(icon: "square.grid.2x2.fill", text: String(localized: "Home-screen widget showing the cheapest nearby", comment: "Onboarding feature row"))
+            featureRow(icon: "car.fill", title: String(localized: "Know if it's worth the drive", comment: "Onboarding feature row title"), subtitle: String(localized: "Factor in your journey and fuel costs to find the best value.", comment: "Onboarding feature row subtitle"))
-            featureRow(icon: "bell.fill", text: String(localized: "Alerts when you approach the cheapest station", comment: "Onboarding feature row"))
+            
            featureRow(icon: "star.fill", title: String(localized: "Keep your favourites close", comment: "Onboarding feature row title"), subtitle: String(localized: "Save stations and quickly compare their latest prices.", comment: "Onboarding feature row subtitle"))
            Spacer()
        }
@@ -155,7 +156,7 @@ struct OnboardingView: View {
            permissionStatusLabel(
                granted: prompter.locationGranted,
                denied: prompter.locationDenied,
-                deniedText: String(localized: "Location was denied. You can still browse all stations, but \"nearest\" sorting needs it.", comment: "Onboarding location denied text")
+                deniedText: String(localized: "Location was denied. You can still browse all stations, but \"closest\" sorting needs it.", comment: "Onboarding location denied text")
            )
            .padding(.top, 24)
@@ -298,18 +299,23 @@ struct OnboardingView: View {
        OnboardingPrimaryButton(title: title, action: action)
    }
-    private func featureRow(icon: String, text: String) -> some View {
+    private func featureRow(icon: String, title: String, subtitle: String) -> some View {
-        HStack(spacing: 12) {
+        HStack(alignment: .top, spacing: 12) {
            Image(systemName: icon)
                .font(.system(size: 20, weight: .semibold))
                .foregroundStyle(Color.accentColor)
                .frame(width: 32)
-            Text(text)
+            VStack(alignment: .leading, spacing: 2) {
                Text(title)
                    .font(.subheadline.bold())
                Text(subtitle)
                    .font(.subheadline)
                    .foregroundStyle(.secondary)
            }
            Spacer(minLength: 0)
        }
        .padding(.horizontal, 36)
-        .padding(.vertical, 10)
+        .padding(.vertical, 8)
    }
    private func permissionStatusLabel(granted: Bool, denied: Bool, deniedText: String) -> some View {
@@ -393,6 +399,9 @@ struct AnimatedOnboardingIcon<Content: View>: View {
    let tint: Color
    var motion: Motion = .gentle
    /// The soft circle behind the glyph. The welcome page turns it off: it shows
    /// the app icon itself and reads better with no container around it.
    var showsHalo: Bool = true
    @ViewBuilder var content: Content
    @State private var settled = false
@@ -402,6 +411,7 @@ struct AnimatedOnboardingIcon<Content: View>: View {
    var body: some View {
        ZStack {
            if showsHalo {
                Circle()
                    .fill(tint.opacity(0.10))
                    .frame(width: 108, height: 108)
@@ -412,6 +422,7 @@ struct AnimatedOnboardingIcon<Content: View>: View {
                    .frame(width: 118, height: 118)
                    .scaleEffect(settled ? 1.03 : 0.97)
                    .opacity(settled ? 1.0 : 0.55)
            }
            content
                .scaleEffect(iconScale)
@@ -285,7 +285,7 @@ struct SettingsView: View {
                    } header: {
                        Text("Debug")
                    } footer: {
-                        Text("The first button applies the real criteria — cheapest \(alertsFuel.displayName.lowercased()) station within \(distanceUnit.format(alertsRadiusKM)) of you — and sends the actual station's name, price and coordinates. The second fires with no criteria at all (nearest seller), still carrying a real station, just to verify a notification appears and tapping it opens directions. Cheapest in range mirrors the app list (selected fuel + distance); the in-range count mirrors the alert radius.")
+                        Text("The first button applies the real criteria — cheapest \(alertsFuel.displayName.lowercased()) station within \(distanceUnit.format(alertsRadiusKM)) of you — and sends the actual station's name, price and coordinates. The second fires with no criteria at all (closest seller), still carrying a real station, just to verify a notification appears and tapping it opens directions. Cheapest in range mirrors the app list (selected fuel + distance); the in-range count mirrors the alert radius.")
                    }
                    .onAppear { onRefreshDebugStatus() }
@@ -29,6 +29,13 @@ struct StationDetailView: View {
    var onToggleFavourite: (() -> Void)? = nil
    @Environment(\.dismiss) private var dismiss
    @State private var showingReportIssueConfirmation = false
    /// Transient "Postcode copied" confirmation on the Address row. Cleared by a task so
    /// the caption fades out on its own.
    @State private var showingCopied = false
    @State private var copyResetTask: Task<Void, Never>?
    /// The "How Best Value works" sheet, opened from the row at the bottom of the
    /// Best Value card (the Key sheet opens the same shared view).
    @State private var showBestValueInfo = false
    private let reportIssueURL = URL(string: "https://www.gov.uk/guidance/report-an-error-in-fuel-prices-or-forecourt-details")!
@@ -36,10 +43,61 @@ struct StationDetailView: View {
        CLLocationCoordinate2D(latitude: station.lat, longitude: station.lng)
    }
    /// The row's address text — the same shared line the listing row uses, with
    /// any leading repeat of the station name stripped.
    private var addressLine: String {
        StationAddressLine.display(address: station.address,
                                   postcode: station.postcode,
                                   name: station.name)
    }
    /// The station's postcode, trimmed. Empty means there is nothing to copy and
    /// the Address row stays inert.
    private var postcodeForCopying: String {
        station.postcode.trimmingCharacters(in: .whitespacesAndNewlines)
    }
    /// Opens Apple Maps driving directions — the same destination (and the same
    /// URL) the Directions button below uses, so the map and the button can
    /// never point at two different places.
    private func openDirections() {
        if let url = station.mapsDirectionsURL {
            UIApplication.shared.open(url)
        }
    }
    /// Copies ONLY the postcode, so it can be pasted into another navigation app
    /// without dragging the station name and street along with it. Confirms with
    /// a light haptic and a caption that fades after ~2s.
    private func copyPostcode() {
        let postcode = postcodeForCopying
        guard !postcode.isEmpty else { return }
        UIPasteboard.general.string = postcode
        UINotificationFeedbackGenerator().notificationOccurred(.success)
        withAnimation(.easeInOut(duration: 0.2)) { showingCopied = true }
        copyResetTask?.cancel()
        copyResetTask = Task {
            try? await Task.sleep(nanoseconds: 2_000_000_000)
            guard !Task.isCancelled else { return }
            withAnimation(.easeInOut(duration: 0.3)) { showingCopied = false }
        }
    }
    var body: some View {
        NavigationStack {
            List {
                Section {
                    // Tapping anywhere on the map opens the same Apple Maps
                    // directions the Directions button below opens.
                    //
                    // The tap target is an OVERLAY Button laid ON TOP of the
                    // Map, not a Button wrapping it. A Map runs its own UIKit
                    // view, and inside a List row the wrapped-Map form left the
                    // card untappable — the touch never reached the button. An
                    // overlay sits above the map, owns an explicit rounded-rect
                    // content shape, and cannot be intercepted by MapKit. The
                    // Map itself stays non-interactive: it is a preview, not
                    // something to pan.
                    Map(position: .constant(.region(MKCoordinateRegion(
                        center: stationCoordinate,
                        span: MKCoordinateSpan(latitudeDelta: 0.02, longitudeDelta: 0.02)
@@ -50,16 +108,88 @@ struct StationDetailView: View {
                    .frame(height: 180)
                    .clipShape(RoundedRectangle(cornerRadius: 12))
                    .allowsHitTesting(false)
-                    .listRowInsets(EdgeInsets())
+                    .overlay {
                        Button {
                            openDirections()
                        } label: {
                            Color.clear
                                .contentShape(RoundedRectangle(cornerRadius: 12))
                        }
                        .buttonStyle(.plain)
                        .accessibilityLabel(String(localized: "Directions to \(station.name)",
                                                   comment: "Accessibility label for the tappable map preview in the More info sheet"))
                        .accessibilityHint(String(localized: "Opens Apple Maps with driving directions",
                                                  comment: "Accessibility hint for the tappable map preview in the More info sheet"))
                    }
                    // No hairline between the map and the Address row: the map
                    // already reads as its own block, so the line only added
                    // clutter. The Address/Distance and Distance/actions lines
                    // stay.
                    .listRowSeparator(.hidden, edges: .bottom)
                    // The map, the address, the distance, any notices and the two
                    // station actions all share ONE card (the list's own row
                    // hairlines separate them), so no hand-rolled Divider here —
                    // adding one would double the line.
                    //
                    // The station name is NOT repeated: the sheet header already
                    // shows it.
                    //
                    // The whole row is a copy control: it puts JUST the postcode
                    // on the clipboard so it can be pasted into another
                    // navigation app. The copy glyph sits ON the address line,
                    // and the "Postcode copied" confirmation is a capsule that floats OVER
                    // THIS ROW — it confirms the address, so it must not hover
                    // over the map. It is an overlay, so nothing moves while it is
                    // up, and `allowsHitTesting(false)` keeps the row tappable
                    // through the capsule.
                    Button {
                        copyPostcode()
                    } label: {
                        HStack(alignment: .firstTextBaseline, spacing: 8) {
                            Text("Address")
                            Spacer(minLength: 8)
                            Text(addressLine)
                                .multilineTextAlignment(.trailing)
                                .foregroundStyle(.secondary)
                            // Stays the plain copy glyph — the capsule carries
                            // the confirmation, so swapping this to a checkmark
                            // would say it twice.
                            Image(systemName: "doc.on.doc")
                                .font(.footnote.weight(.semibold))
                                .foregroundStyle(.secondary)
                        }
                    }
                    .buttonStyle(.plain)
                    .disabled(postcodeForCopying.isEmpty)
                    .accessibilityLabel(String(localized: "Address: \(addressLine)",
                                               comment: "Accessibility label for the address row in the More info sheet"))
                    .accessibilityHint(postcodeForCopying.isEmpty
                        ? ""
                        : String(localized: "Copies the postcode",
                                 comment: "Accessibility hint for the address row in the More info sheet"))
                    // Horizontally centred over the address line itself.
                    .overlay(alignment: .center) {
                        if showingCopied {
                            HStack(spacing: 4) {
                                Image(systemName: "checkmark")
                                Text("Postcode copied")
                            }
                            .font(.footnote.weight(.semibold))
                            .foregroundStyle(.green)
                            .padding(.horizontal, 10)
                            .padding(.vertical, 6)
                            // A material base keeps it legible where the tint
                            // alone (the notice-capsule recipe) washes out.
                            .background {
                                Capsule()
                                    .fill(.regularMaterial)
                                    .overlay(Capsule().fill(Color.green.opacity(0.22)))
                            }
                            .allowsHitTesting(false)
                            .accessibilityHidden(true)
                            .transition(.opacity)
                        }
                    }
                Section {
                    // The station name is NOT repeated here: the sheet header
                    // (navigationTitle, inline) already shows it, and the address
                    // line below has any leading repeat of the name stripped, so
                    // a "Name" row only said it twice.
                    LabeledContent("Address", value: StationAddressLine.display(address: station.address,
                                                                             postcode: station.postcode,
                                                                             name: station.name))
                    if let location {
                        let distance = FuelStore.stationDistanceDisplay(
                            station: station, userLat: location.lat, userLng: location.lng
@@ -75,6 +205,28 @@ struct StationDetailView: View {
                            .padding(.vertical, 6)
                            .background(Capsule().fill(Color.orange.opacity(0.14)))
                    }
                    // The two station actions are the LAST row of the card: half
                    // the width each, on one line, below Address and Distance.
                    HStack(spacing: 12) {
                        Button {
                            openDirections()
                        } label: {
                            Label("Directions", systemImage: "arrow.triangle.turn.up.right.diamond.fill")
                                .frame(maxWidth: .infinity)
                        }
                        Button(role: .destructive) {
                            showingReportIssueConfirmation = true
                        } label: {
                            Label("Report issue", systemImage: "exclamationmark.bubble.fill")
                                .frame(maxWidth: .infinity)
                        }
                    }
                    // REQUIRED: with the default button style a `List` row has ONE
                    // row-wide tap target, so two buttons in the same row BOTH fire
                    // on any tap inside the row (tapping Directions also showed the
                    // report-issue alert and vice versa). `.borderless` gives each
                    // button its own hit area — the half-row frame above.
                    .buttonStyle(.borderless)
                }
                // Only shown while Fuel Savings backs it. `fuelSavings` is nil
                // when the feature is off — `savingsLookup` returns nil when it
@@ -86,7 +238,8 @@ struct StationDetailView: View {
                            availability: fuelSavings,
                            amount: fuelSavingsAmount,
                            baselineStationName: savingsBaselineStationName,
-                            distanceUnit: distanceUnit
+                            distanceUnit: distanceUnit,
                            showInfoAction: { showBestValueInfo = true }
                        )
                    } header: {
                        Text("Best Value")
@@ -105,20 +258,6 @@ struct StationDetailView: View {
                }
 #endif
                Section {
                    Button {
                        if let url = station.mapsDirectionsURL {
                            UIApplication.shared.open(url)
                        }
                    } label: {
                        Label("Directions", systemImage: "arrow.triangle.turn.up.right.diamond.fill")
                    }
                    Button(role: .destructive) {
                        showingReportIssueConfirmation = true
                    } label: {
                        Label("Report issue", systemImage: "exclamationmark.bubble.fill")
                    }
                }
            }
            .navigationTitle(station.name)
            .navigationBarTitleDisplayMode(.inline)
@@ -130,6 +269,8 @@ struct StationDetailView: View {
            } message: {
                Text("Are you sure you want to report an error in fuel prices or forecourt details? This will open in your chosen browser.")
            }
            // Nested sheet: the More info sheet stays put behind the explanation.
            .sheet(isPresented: $showBestValueInfo) { HowBestValueSheet() }
            .toolbar {
                ToolbarItem(placement: .cancellationAction) {
                    Button("Done") { dismiss() }
@@ -149,6 +290,18 @@ struct StationDetailView: View {
                    )
                }
            }
            // The "Postcode copied" fade-out task must not outlive the sheet.
            .onDisappear { copyResetTask?.cancel() }
 #if DEBUG
            // QA hook: `-qaCopyPostcode` runs the REAL copy on open, so the
            // clipboard value (read back with `xcrun simctl pbpaste`) and the
            // "Postcode copied" confirmation can be verified without a tap — simctl
            // cannot drive taps into the Simulator.
            .task {
                guard ProcessInfo.processInfo.arguments.contains("-qaCopyPostcode") else { return }
                copyPostcode()
            }
 #endif
        }
    }
 }
@@ -19,7 +19,6 @@ struct StationsView: View {
    let fuelSavingsSettings: FuelSavingsSettings
    @Binding var fuelSavingsAmount: FuelSavingsFillAmount
    var savingsForStation: (FuelStation) -> FuelSavingsAvailability? = { _ in nil }
    var savingsBaselineStationName: String?
    var journeyDiagnosticsForStation: (FuelStation) -> FuelSavingsJourneyDiagnostics? = { _ in nil }
    var onToggleFavourite: (FuelStation, FuelType) -> Void = { _, _ in }
    var onRefresh: () async -> Void = {}
@@ -35,6 +34,11 @@ struct StationsView: View {
    /// info button in the header — keeps the list focused on stations.
    @State private var showKey = false
    /// The "How Best Value works" sheet. The explanation used to be a section in
    /// the Key sheet; it is its own sheet now, opened from a row at the bottom of
    /// the Key sheet and from the More info sheet.
    @State private var showBestValueInfo = false
    /// Station picked for the More-info sheet (tap action = showDetails).
    @State private var detailStation: FuelStation?
@@ -63,7 +67,7 @@ struct StationsView: View {
        }
        if location != nil {
            if sortMode == .closest {
-                return "\(mode) \(fuel) stations — nearest first, best value within \(miles) \(unit) · \(ratio) stations updated"
+                return "\(mode) \(fuel) stations — closest first, best value within \(miles) \(unit) · \(ratio) stations updated"
            }
            return "\(mode) \(fuel) within \(miles) \(unit) — \(ratio) stations updated · \(tapHint)"
        }
@@ -89,7 +93,7 @@ struct StationsView: View {
                    }
                    .pickerStyle(.segmented)
                    // In Closest mode the whole country is the pool — the
-                    // radius is meaningless ("nearest" must never return an
+                    // radius is meaningless ("closest" must never return an
                    // empty state), so the picker is disabled but its value is
                    // kept for when the user switches back to Cheapest.
                    .disabled(sortMode == .closest)
@@ -218,6 +222,18 @@ struct StationsView: View {
                if ProcessInfo.processInfo.arguments.contains("-showKeySheet") {
                    showKey = true
                }
                // QA hook: `-showBestValueInfo` opens the Key sheet and then the
                // explanation on top of it, so the copy can be screenshotted
                // without a tap. It has to chain: the explanation's `.sheet` lives
                // inside the Key sheet's content, so it cannot present while the
                // Key sheet is shut.
                if BestValueInfoQA.isRequested {
                    showKey = true
                    Task { @MainActor in
                        try? await Task.sleep(nanoseconds: 1_200_000_000)
                        showBestValueInfo = true
                    }
                }
 #endif
            }
            .task(id: stations.count) {
@@ -250,14 +266,14 @@ struct StationsView: View {
                                keyCard(
                                    icon: "checkmark.circle.fill", color: .green,
                                    title: "Good price", blurb: "Within 1.5p of the cheapest",
-                                    threshold: "≤1.5p")
+                                    threshold: "0–1.5p")
                                keyCard(
                                    icon: "equal.circle.fill", color: .orange,
-                                    title: "Okay", blurb: "Within 4p of the cheapest",
+                                    title: "Okay", blurb: "More than 1.5p, up to 4p above the cheapest",
-                                    threshold: "≤4p")
+                                    threshold: ">1.5–4p")
                                keyCard(
                                    icon: "exclamationmark.circle.fill", color: .red,
-                                    title: "Pricey", blurb: "More than 4p over the cheapest",
+                                    title: "Pricey", blurb: "More than 4p above the cheapest",
                                    threshold: ">4p")
                                Rectangle()
                                    .fill(Color(.separator))
@@ -289,25 +305,26 @@ struct StationsView: View {
                                .padding(.horizontal, 14)
                                .padding(.vertical, 12)
                                .keyCardFill(color: .yellow)
-                                Text("Colours match each station's rating on the list")
+                                Text("Price differences are shown in pence per litre (p/litre). Colours match each station's rating on the list.")
                                    .font(.footnote)
                                    .foregroundStyle(.secondary)
-                                    .frame(maxWidth: .infinity)
+                                    .frame(maxWidth: .infinity, alignment: .center)
                                    .multilineTextAlignment(.center)
                                    .padding(.top, 4)
                            }
                            .listRowBackground(Color.clear)
                            .listRowInsets(EdgeInsets(top: 4, leading: 16, bottom: 4, trailing: 16))
                        }
-                        Section("How Best Value works") {
+                        // The explanation is its own sheet now, opened from this
-                            VStack(alignment: .leading, spacing: 10) {
+                        // row. Gated exactly like the More info row and the Best
-                                Text("Best Value compares each station with the closest one. It works out what you could save at the pump, then takes off the fuel used for the extra return journey.")
+                        // Value card itself: `isConfigured` is the same condition
-                                Text("Your fill-up amount and MPG are used to show whether you could save overall.")
+                        // `savingsForStation` uses, so the row appears and
-                                Text("Results are marked “Approx.” until road distances are available.")
+                        // disappears with the feature.
-                                    .foregroundStyle(.secondary)
+                        if fuelSavingsSettings.isConfigured {
                            Section {
                                HowBestValueRow { showBestValueInfo = true }
                            }
                            .font(.subheadline)
                            .padding(.vertical, 4)
                        }
                    }
                    .navigationTitle("Key")
@@ -317,6 +334,8 @@ struct StationsView: View {
                            Button("Done") { showKey = false }
                        }
                    }
                    // Nested sheet: the Key sheet stays behind the explanation.
                    .sheet(isPresented: $showBestValueInfo) { HowBestValueSheet() }
                }
                .presentationDetents([.fraction(0.6)])
                .presentationBackground(Color(UIColor.systemGroupedBackground))
@@ -328,7 +347,6 @@ struct StationsView: View {
                    distanceUnit: distanceUnit,
                    fuelSavings: savingsForStation(station),
                    fuelSavingsAmount: fuelSavingsAmount,
                    savingsBaselineStationName: savingsBaselineStationName,
                    journeyDiagnostics: journeyDiagnosticsForStation(station),
                    isFavourite: favouriteIDs.contains(station.id),
                    onToggleFavourite: { onToggleFavourite(station, selectedFuel) }
@@ -358,6 +376,7 @@ struct StationsView: View {
                Text(LocalizedStringKey(blurb))
                    .font(.caption)
                    .foregroundStyle(.secondary)
                    .fixedSize(horizontal: false, vertical: true)
            }
            Spacer(minLength: 8)
            HStack(spacing: 6) {
@@ -76,7 +76,7 @@ struct WidgetMockScreen: View {
        let location = FuelStore.loadLocation().map { Coordinate(lat: $0.lat, lng: $0.lng) }
        // 5 miles — the widget config's default Distance. WidgetDistance itself
        // lives in the widget target, so the preview mirrors its default here.
-        let radiusKM = 5 * 1.60934
+        let radiusKM = 5 * FuelSavingsCalculator.kilometresPerMile
        let all = FuelStore.loadStations()
        let selling = all.filter { station in
            guard station.prices[fuel] != nil else { return false }
@@ -94,7 +94,8 @@ struct WidgetMockScreen: View {
            isFavourites: false,
            location: location,
            locationSource: location != nil ? "live" : "none",
-            unit: FuelStore.loadDistanceUnit()
+            unit: FuelStore.loadDistanceUnit(),
            bestValueResult: nil
        )
    }
 }
@@ -12,7 +12,7 @@
 "TOP" = "TOP";
 /* Stations tab */
-"Closest %@ stations — nearest first, best value within %lld %@ · %lld stations" = "Closest %@ stations — nearest first, best value within %lld %@ · %lld stations";
+"Closest %@ stations — closest first, best value within %lld %@ · %lld stations" = "Closest %@ stations — closest first, best value within %lld %@ · %lld stations";
 "Cheapest %@ within %lld %@ — %lld stations · tap a station for directions." = "Cheapest %@ within %lld %@ — %lld stations · tap a station for directions.";
 "%@ %@ — tap a station for directions." = "%@ %@ — tap a station for directions.";
 "Fuel type" = "Fuel type";
@@ -26,15 +26,15 @@
 "Good price" = "Good price";
 "Within 1.5p of the cheapest" = "Within 1.5p of the cheapest";
 "Okay" = "Okay";
-"Within 4p of the cheapest" = "Within 4p of the cheapest";
+"More than 1.5p, up to 4p above the cheapest" = "More than 1.5p, up to 4p above the cheapest";
 "Pricey" = "Pricey";
-"More than 4p over the cheapest" = "More than 4p over the cheapest";
+"More than 4p above the cheapest" = "More than 4p above the cheapest";
 "Top result for the current sort" = "Top result for the current sort";
 "Star a station to add it to Favourites" = "Star a station to add it to Favourites";
 /* Star button in the More info sheet heading (accessibility labels) */
 "Add to favourites" = "Add to favourites";
 "Remove from favourites" = "Remove from favourites";
-"Colours match each station's rating on the list" = "Colours match each station's rating on the list";
+"Price differences are shown in pence per litre (p/litre). Colours match each station's rating on the list." = "Price differences are shown in pence per litre (p/litre). Colours match each station's rating on the list.";
 "How Best Value works" = "How Best Value works";
 "Best Value compares each station with the closest one. It works out what you could save at the pump, then takes off the fuel used for the extra return journey." = "Best Value compares each station with the closest one. It works out what you could save at the pump, then takes off the fuel used for the extra return journey.";
 "Your fill-up amount and MPG are used to show whether you could save overall." = "Your fill-up amount and MPG are used to show whether you could save overall.";
@@ -71,12 +71,17 @@
 "Name" = "Name";
 "Address" = "Address";
 "Distance" = "Distance";
 "Copies the postcode" = "Copies the postcode";
 "Postcode copied" = "Postcode copied";
 "Address: %@" = "Address: %@";
 "Directions to %@" = "Directions to %@";
 "Opens Apple Maps with driving directions" = "Opens Apple Maps with driving directions";
 "Show introduction" = "Replay onboarding";
 "Replay the welcome screen, including the location and notification permission prompts." = "Replay the welcome screen, including the location and notification permission prompts.";
 "Test alert notification (real data)" = "Test alert notification (real data)";
 "Send plain test notification" = "Send plain test notification";
 "Debug" = "Debug";
-"The first button applies the real criteria — cheapest %@ station within %@ of you — and sends the actual station's name, price and coordinates. The second fires with no criteria at all (nearest seller), still carrying a real station, just to verify a notification appears and tapping it opens directions. Cheapest in range mirrors the app list (selected fuel + distance); the in-range count mirrors the alert radius." = "The first button applies the real criteria — cheapest %@ station within %@ of you — and sends the actual station's name, price and coordinates. The second fires with no criteria at all (nearest seller), still carrying a real station, just to verify a notification appears and tapping it opens directions. Cheapest in range mirrors the app list (selected fuel + distance); the in-range count mirrors the alert radius.";
+"The first button applies the real criteria — cheapest %@ station within %@ of you — and sends the actual station's name, price and coordinates. The second fires with no criteria at all (closest seller), still carrying a real station, just to verify a notification appears and tapping it opens directions. Cheapest in range mirrors the app list (selected fuel + distance); the in-range count mirrors the alert radius." = "The first button applies the real criteria — cheapest %@ station within %@ of you — and sends the actual station's name, price and coordinates. The second fires with no criteria at all (closest seller), still carrying a real station, just to verify a notification appears and tapping it opens directions. Cheapest in range mirrors the app list (selected fuel + distance); the in-range count mirrors the alert radius.";
 "Widget Diagnostics" = "Widget Diagnostics";
 "Each beacon is written by the widget extension at the end of every timeline entry for that intent type (keychain, so it survives free-SideStore installs). A missing SMALL beacon after adding the small widget means its timeline never reaches the provider. Installed widgets come from WidgetCenter." = "Each beacon is written by the widget extension at the end of every timeline entry for that intent type (keychain, so it survives free-SideStore installs). A missing SMALL beacon after adding the small widget means its timeline never reaches the provider. Installed widgets come from WidgetCenter.";
 "SMALL: %@" = "SMALL: %@";
@@ -134,7 +139,7 @@
 "Connection shows the source of the current prices (CSV mirror or API). Data updated is the latest price change reported by the GOV.UK server itself. Prices use the UK Fuel Finder dataset (Open Government Licence) and refresh up to twice a day." = "Connection shows the source of the current prices (CSV mirror or API). Data updated is the latest price change reported by the GOV.UK server itself. Prices use the UK Fuel Finder dataset (Open Government Licence) and refresh up to twice a day.";
 "Prices near you" = "Prices near you";
 "FuelBoard uses your location to show prices around you and rank stations by distance. Your position never leaves the device." = "FuelBoard uses your location to show prices around you and rank stations by distance. Your position never leaves the device.";
-"Location was denied. You can still browse all stations, but \"nearest\" sorting needs it." = "Location was denied. You can still browse all stations, but \"nearest\" sorting needs it.";
+"Location was denied. You can still browse all stations, but \"closest\" sorting needs it." = "Location was denied. You can still browse all stations, but \"closest\" sorting needs it.";
 "Allowed" = "Allowed";
 "The system prompt will appear next." = "The system prompt will appear next.";
 "FuelBoard can notify you when you're approaching the cheapest station in your alert radius — even with the app closed. You can switch this off in the Alerts tab." = "FuelBoard can notify you when you're approaching the cheapest station in your alert radius — even with the app closed. You can switch this off in the Alerts tab.";
@@ -211,10 +211,10 @@ final class DistanceTests: XCTestCase {
    }
    func testMilesConversion() {
-        // 5 miles ≈ 8.05 km (app uses × 1.60934)
+        // 5 miles ≈ 8.05 km using the shared exact conversion
-        XCTAssertEqual(5 * 1.60934, 8.0467, accuracy: 0.001)
+        XCTAssertEqual(5 * FuelSavingsCalculator.kilometresPerMile, 8.04672, accuracy: 0.001)
-        // 262 km ≈ 162.8 miles (display uses × 0.621371)
+        // 262 km ≈ 162.8 miles using the shared exact conversion
-        XCTAssertEqual(262 * 0.621371, 162.8, accuracy: 0.1)
+        XCTAssertEqual(262 / FuelSavingsCalculator.kilometresPerMile, 162.8, accuracy: 0.1)
    }
 }
@@ -267,14 +267,14 @@ final class FuelTypeLabelTests: XCTestCase {
    }
    func testDistanceUnitConversion() {
-        XCTAssertEqual(DistanceUnit.miles.toKM(1), 1.60934, accuracy: 0.00001)
+        XCTAssertEqual(DistanceUnit.miles.toKM(1), 1.609344, accuracy: 0.00001)
        XCTAssertEqual(DistanceUnit.kilometers.toKM(5), 5)
-        XCTAssertEqual(DistanceUnit.miles.fromKM(1.60934), 1, accuracy: 0.00001)
+        XCTAssertEqual(DistanceUnit.miles.fromKM(1.609344), 1, accuracy: 0.00001)
        XCTAssertEqual(DistanceUnit.kilometers.fromKM(2.5), 2.5)
    }
    func testDistanceUnitFormat() {
-        XCTAssertEqual(DistanceUnit.miles.format(1.60934), "1.0 mi")
+        XCTAssertEqual(DistanceUnit.miles.format(1.609344), "1.0 mi")
        XCTAssertEqual(DistanceUnit.kilometers.format(3.4), "3.4 km")
    }
 }
@@ -400,7 +400,7 @@ final class AlertRadiusFollowTests: XCTestCase {
        FuelStore.saveDistanceUnit(.miles)
        FuelStore.saveStationLimit(5)
        let eff = FuelStore.effectiveAlertsRadiusKM(followsSearch: true, manualKM: 3)
-        XCTAssertEqual(eff, 5 * 1.60934, accuracy: 0.001)
+        XCTAssertEqual(eff, 5 * FuelSavingsCalculator.kilometresPerMile, accuracy: 0.001)
    }
 }
@@ -731,7 +731,7 @@ final class RoadDistanceCacheTests: XCTestCase {
            sourceLat: sourceLat, sourceLng: sourceLng,
            entries: ["far": .init(meters: 23_500, lat: 51.5074, lng: -0.1278, returnMeters: 23_500)])
-        let tenMilesKM = 10 * 1.60934
+        let tenMilesKM = 10 * FuelSavingsCalculator.kilometresPerMile
        XCTAssertLessThan(far.distanceKM(to: sourceLat, lng2: sourceLng), tenMilesKM,
                          "fixture: the straight line must sit INSIDE the 10-mile radius")
        XCTAssertGreaterThan(FuelStore.displayDistanceKM(station: far, userLat: sourceLat, userLng: sourceLng),
@@ -743,7 +743,7 @@ final class RoadDistanceCacheTests: XCTestCase {
                                                      userLat: sourceLat, userLng: sourceLng),
                       "a station whose row reads 14.6 mi must not appear in a 10-mile list")
        // The same station is back once the radius actually covers its distance.
-        XCTAssertTrue(FuelStore.isWithinDisplayRadius(station: far, radiusKM: 15 * 1.60934,
+        XCTAssertTrue(FuelStore.isWithinDisplayRadius(station: far, radiusKM: 15 * FuelSavingsCalculator.kilometresPerMile,
                                                     userLat: sourceLat, userLng: sourceLng))
        // …and an unknown station (no cached route) still tests its displayed
        // straight line, so nothing changes for the bulk of the list.
@@ -783,10 +783,10 @@ final class RoadDistanceCacheTests: XCTestCase {
        XCTAssertFalse(confirmed.isApproximate, "a cached road distance is not an approximation")
        XCTAssertEqual(confirmed.km, 3.2, accuracy: 0.0001)
        XCTAssertEqual(confirmed.label(unit: .miles), "2.0 mi")
-        XCTAssertEqual(confirmed.labelWithReturn(unit: .miles), "2.0 mi (4.0 mi return)")
+        XCTAssertEqual(confirmed.labelWithReturn(unit: .miles), "2.0 mi (4.2 mi return)")
-        XCTAssertEqual(confirmed.labelWithReturn(unit: .kilometers), "3.2 km (6.4 km return)")
+        XCTAssertEqual(confirmed.labelWithReturn(unit: .kilometers), "3.2 km (6.8 km return)")
        XCTAssertEqual(confirmed.spokenLabel(unit: .miles), "2.0 miles")
-        XCTAssertEqual(confirmed.spokenLabelWithReturn(unit: .miles), "2.0 miles, 4.0 miles return")
+        XCTAssertEqual(confirmed.spokenLabelWithReturn(unit: .miles), "2.0 miles, 4.2 miles return")
        let approximate = FuelStore.stationDistanceDisplay(station: unrouted, userLat: 51.6, userLng: -0.1)
        XCTAssertTrue(approximate.isApproximate, "the straight-line fallback must be identified as one")
@@ -794,11 +794,40 @@ final class RoadDistanceCacheTests: XCTestCase {
        XCTAssertEqual(approximate.label(unit: .miles),
                       "Approx. " + DistanceUnit.miles.format(approximate.km))
        XCTAssertEqual(approximate.labelWithReturn(unit: .miles),
-                       "Approx. \(DistanceUnit.miles.format(approximate.km)) (\(DistanceUnit.miles.format(approximate.km * 2)) return)")
+                       "Approx. " + DistanceUnit.miles.format(approximate.km) + " (Approx. " + DistanceUnit.miles.format(approximate.km * 2) + " return)")
        XCTAssertEqual(approximate.spokenLabel(unit: .kilometers),
                       "Approximately " + DistanceUnit.kilometers.spoken(approximate.km))
        XCTAssertEqual(approximate.spokenLabelWithReturn(unit: .kilometers),
-                       "Approximately \(DistanceUnit.kilometers.spoken(approximate.km)), \(DistanceUnit.kilometers.spoken(approximate.km * 2)) return")
+                       "Approximately \(DistanceUnit.kilometers.spoken(approximate.km)), Approximately \(DistanceUnit.kilometers.spoken(approximate.km * 2)) return")
    }
    func testDisplayedReturnUsesTheIndependentlyMeasuredReturnRoute() {
        FuelStore.setAccurateDistancesEnabled(true)
        let station = station("separate-legs", 51.5074, -0.1278)
        FuelStore.saveRoadDistances(
            sourceLat: 51.6,
            sourceLng: -0.1,
            entries: ["separate-legs": .init(
                meters: 4_650,
                lat: station.lat,
                lng: station.lng,
                returnMeters: 4_820
            )],
            complete: true
        )
        let display = FuelStore.stationDistanceDisplay(
            station: station,
            userLat: 51.6,
            userLng: -0.1
        )
        XCTAssertEqual(display.km, 4.65, accuracy: 1e-12)
        XCTAssertEqual(display.returnKM, 9.47, accuracy: 1e-12)
        XCTAssertFalse(display.isApproximate)
        XCTAssertFalse(display.returnIsApproximate)
        XCTAssertEqual(display.labelWithReturn(unit: .miles), "2.9 mi (5.9 mi return)")
        XCTAssertEqual(display.spokenLabelWithReturn(unit: .miles), "2.9 miles, 5.9 miles return")
    }
    /// The marker tracks the cache on its own: same station, bare figure once its
@@ -961,7 +990,7 @@ final class RoadDistanceCacheTests: XCTestCase {
        let display = FuelStore.stationDistanceDisplay(station: routed, userLat: 51.6, userLng: -0.1)
        XCTAssertTrue(display.isApproximate, "the browse layer must never serve routes with the mode off")
        XCTAssertEqual(display.km, routed.distanceKM(to: 51.6, lng2: -0.1), accuracy: 0.0001)
-        XCTAssertTrue(FuelStore.isWithinDisplayRadius(station: routed, radiusKM: 10 * 1.60934,
+        XCTAssertTrue(FuelStore.isWithinDisplayRadius(station: routed, radiusKM: 10 * FuelSavingsCalculator.kilometresPerMile,
                                                      userLat: 51.6, userLng: -0.1),
                      "with the mode off the radius is a straight-line radius, agreeing with the rows")
@@ -1000,7 +1029,7 @@ final class RoadDistanceCacheTests: XCTestCase {
        let display = FuelStore.stationDistanceDisplay(station: routed, userLat: 51.6, userLng: -0.1)
        XCTAssertFalse(display.isApproximate)
        XCTAssertEqual(display.km, 23.5, accuracy: 0.0001)
-        XCTAssertFalse(FuelStore.isWithinDisplayRadius(station: routed, radiusKM: 10 * 1.60934,
+        XCTAssertFalse(FuelStore.isWithinDisplayRadius(station: routed, radiusKM: 10 * FuelSavingsCalculator.kilometresPerMile,
                                                       userLat: 51.6, userLng: -0.1),
                       "in this mode a 10-mile radius is 10 miles of driving, matching the routed rows")
    }
@@ -263,6 +263,23 @@ final class FuelSavingsCalculationAuditTests: XCTestCase {
        XCTAssertEqual(FuelSavingsFormat.money(0), "£0.00")
    }
    func testIndependentlyRoundedFinancialComponentsMayDifferFromRoundedTotalByOnePenny() {
        let gross = 1.234
        let drivingCost = 0.005
        let net = gross - drivingCost
        XCTAssertEqual(FuelSavingsFormat.money(gross), "£1.23")
        XCTAssertEqual(FuelSavingsFormat.money(drivingCost), "£0.01")
        XCTAssertEqual(FuelSavingsFormat.money(net), "£1.23")
        let independentlyRoundedSubtraction = 1.23 - 0.01
        XCTAssertEqual(independentlyRoundedSubtraction, 1.22, accuracy: 1e-12)
        XCTAssertNotEqual(
            FuelSavingsFormat.money(independentlyRoundedSubtraction),
            FuelSavingsFormat.money(net),
            "the raw total remains authoritative; independently rounded components need not add back exactly"
        )
    }
    func testSortingUsesTheUnroundedNetSoRoundedTiesStillOrderCorrectly() {
        // Two rows that both DISPLAY "£0.18 worse off" but differ in the raw
        // figure: the sort must use the raw value, never the shown one.
@@ -454,6 +454,25 @@ final class FuelSavingsEstimateTests: XCTestCase {
    // MARK: Row copy
    func testCompactComparisonLabelUsesGenericNearestStationWording() {
        XCTAssertEqual(
            FuelSavingsFormat.comparisonLabel(status: .worthTheDrive),
            "vs closest station"
        )
        XCTAssertEqual(
            FuelSavingsFormat.comparisonLabel(status: .notWorthTheDrive),
            "vs closest station"
        )
        XCTAssertEqual(
            FuelSavingsFormat.comparisonLabel(status: .breakEven),
            "vs closest station"
        )
        XCTAssertEqual(
            FuelSavingsFormat.comparisonLabel(status: .baseline),
            "your baseline"
        )
    }
    /// The row's extra-driving caption must follow the DISPLAY unit: the stored
    /// figure is always miles, and converting by the current unit instead would
    /// leave a km user's figure unlabelled and 1.6× too large.
@@ -486,7 +505,7 @@ final class FuelSavingsEstimateTests: XCTestCase {
            netSavingGBP: 0,
            status: .baseline
        )
-        XCTAssertEqual(baseline.capsuleHeadline(), "Closest station · your baseline")
+        XCTAssertEqual(baseline.capsuleHeadline(), "Closest station")
        let saving = FuelSavingsResult(
            fillLitres: 30,
@@ -51,7 +51,7 @@ private enum WatchDistanceUnit: String, Codable {
    func format(km: Double) -> String {
        switch self {
        case .miles:
-            return String(format: "%.1f mi", km * 0.621371)
+            return String(format: "%.1f mi", km / 1.609344)
        case .kilometers:
            return String(format: "%.1f km", km)
        }
@@ -95,7 +95,8 @@ struct FuelPriceTimelineProvider<Configuration: WidgetConfigurationIntent & Widg
            data: FuelPriceWidgetEntryData(
                date: Date(), stations: Array(sample.prefix(8)),
                fuel: .e10, sort: .cheapest, isFavourites: false,
-                location: nil, locationSource: "none", unit: unit
+                location: nil, locationSource: "none", unit: unit,
                bestValueResult: nil
            )
        )
    }
@@ -146,8 +147,9 @@ struct FuelPriceTimelineProvider<Configuration: WidgetConfigurationIntent & Widg
        // Per-widget config: fuel + sort + distance come from THIS widget instance.
        let fuel = FuelType(rawValue: configuration.fuel.rawValue) ?? .e10
        let isFavourites = configuration.sort == .favourites
-        let sort: SortMode = configuration.sort == .closest ? .closest : .cheapest
+        let sort: SortMode = configuration.sort == .closest ? .closest :
-        let radiusKM = Double(configuration.distance.miles) * 1.60934
+            (configuration.sort == .worthTheDrive ? .worthTheDrive : .cheapest)
        let radiusKM = Double(configuration.distance.miles) * FuelSavingsCalculator.kilometresPerMile
        // 1) Try a fresh location fix (bounded to a few seconds). The fix's own
        // accuracy is kept: the extension runs its own CLLocationManager and is
@@ -200,6 +202,7 @@ struct FuelPriceTimelineProvider<Configuration: WidgetConfigurationIntent & Widg
        // 3) Load + order stations per widget mode.
        let ordered: [FuelStation]
        var bestValueResult: FuelSavingsResult?
        if isFavourites {
            let favourites = FuelStore.loadFavourites()
                .filter { $0.station.prices[$0.fuel] != nil }
@@ -265,7 +268,7 @@ struct FuelPriceTimelineProvider<Configuration: WidgetConfigurationIntent & Widg
            let filtered = stations.filter { $0.prices[fuel] != nil }
            switch sort {
            case .closest:
-                // Nearest first; price only breaks ties.
+                // Closest first; price only breaks ties.
                ordered = filtered.sorted { lhs, rhs in
                    if let origin {
                        let lDist = FuelStore.displayDistanceKM(station: lhs, userLat: origin.lat, userLng: origin.lng)
@@ -274,9 +277,59 @@ struct FuelPriceTimelineProvider<Configuration: WidgetConfigurationIntent & Widg
                    }
                    return lhs.prices[fuel]! < rhs.prices[fuel]!
                }
-            case .cheapest, .worthTheDrive:
+            case .worthTheDrive:
-                // Cheapest first; distance only breaks ties. Worth-the-Drive is app-only;
+                guard let origin else {
-                // widgets fall back to cheapest ordering if a stored config ever carries it.
+                    ordered = filtered.sorted { $0.prices[fuel]! < $1.prices[fuel]! }
                    break
                }
                let journey: (FuelStation) -> FuelSavingsJourney? = { station in
                    if let meters = FuelStore.roadJourneyMeters(for: station, userLat: origin.lat, userLng: origin.lng) {
                        let confidence: JourneyConfidence = FuelStore.hasMeasuredReturnRoadDistance(
                            for: station, userLat: origin.lat, userLng: origin.lng
                        ) ? .confirmed : .estimated
                        return FuelSavingsJourney(
                            miles: meters / FuelSavingsCalculator.metresPerMile,
                            basis: .roadRoute,
                            confidence: confidence
                        )
                    }
                    return FuelSavingsJourney(
                        miles: FuelStore.displayDistanceKM(station: station, userLat: origin.lat, userLng: origin.lng) /
                            FuelSavingsCalculator.kilometresPerMile * 2,
                        basis: .straightLine,
                        confidence: .estimated
                    )
                }
                let settings = FuelStore.loadFuelSavingsSettings()
                guard settings.isConfigured else {
                    ordered = filtered.sorted { $0.prices[fuel]! < $1.prices[fuel]! }
                    break
                }
                let amount = settings.preferredAmount
                let baseline = FuelSavingsEstimator.baselineStation(filtered, fuel: fuel, journeyMiles: journey)
                let ranked = FuelSavingsEstimator.ranked(
                    filtered,
                    fuel: fuel,
                    baseline: baseline,
                    settings: settings,
                    amount: amount,
                    journeyMiles: journey
                )
                ordered = ranked
                if let first = ranked.first, let baseline {
                    let baselineJourney = journey(baseline)
                    let result = FuelSavingsEstimator.availability(
                        station: first,
                        fuel: fuel,
                        baseline: baseline,
                        settings: settings,
                        amount: amount,
                        journey: journey(first),
                        baselineJourney: baselineJourney
                    )
                    bestValueResult = result?.result
                }
            case .cheapest:
                ordered = filtered.sorted { lhs, rhs in
                    let lPrice = lhs.prices[fuel]!
                    let rPrice = rhs.prices[fuel]!
@@ -295,7 +348,7 @@ struct FuelPriceTimelineProvider<Configuration: WidgetConfigurationIntent & Widg
                date: Date(), stations: Array(ordered.prefix(8)),
                fuel: fuel, sort: sort, isFavourites: isFavourites,
                location: origin, locationSource: source,
-                unit: unit
+                unit: unit, bestValueResult: bestValueResult
            )
        ), originSource)
    }
@@ -30,6 +30,7 @@ enum WidgetFuel: String, AppEnum, CaseIterable, Codable {
 enum WidgetSort: String, AppEnum, CaseIterable, Codable {
    case cheapest
    case closest
    case worthTheDrive
    case favourites
    static var typeDisplayRepresentation: TypeDisplayRepresentation = "Sort by"
@@ -37,6 +38,7 @@ enum WidgetSort: String, AppEnum, CaseIterable, Codable {
    static var caseDisplayRepresentations: [WidgetSort: DisplayRepresentation] = [
        .cheapest: "Cheapest",
        .closest: "Closest",
        .worthTheDrive: "Best Value",
        .favourites: "Favourites",
    ]
 }
@@ -111,6 +113,13 @@ struct FuelBoardWidgetConfigurationIntent: WidgetConfigurationIntent, WidgetConf
                    \.$fuel
                    \.$sort
                }
            } otherwise: {
                When(\.$sort, .equalTo, WidgetSort.worthTheDrive) {
                    Summary("Show \(\.$fuel) by \(\.$sort) within \(\.$distance)") {
                        \.$fuel
                        \.$sort
                        \.$distance
                    }
                } otherwise: {
                    Summary("Show \(\.$fuel) by \(\.$sort)") {
                        \.$fuel
@@ -119,4 +128,5 @@ struct FuelBoardWidgetConfigurationIntent: WidgetConfigurationIntent, WidgetConf
                }
            }
        }
    }
 }
@@ -29,11 +29,12 @@ struct FuelPriceWidgetEntryData {
    let date: Date
    let stations: [FuelStation]   // already filtered + sorted per config
    let fuel: FuelType
-    let sort: SortMode            // per-widget: cheapest | closest
+    let sort: SortMode            // per-widget: cheapest | closest | Best Value
-    let isFavourites: Bool        // favourites mode: pinned stations, cheapest-first
+    let isFavourites: Bool        // favourites mode: pinned stations
    let location: Coordinate?
    let locationSource: String    // "live" | "cached" | "none"
    let unit: DistanceUnit        // user's display unit for distances
    let bestValueResult: FuelSavingsResult? // only populated for the small Best Value face
 }
 /// The widget face, parameterised by family so the app can preview it.
@@ -67,9 +68,11 @@ struct FuelPriceWidgetContent: View {
    private var heading: String {
        if entry.isFavourites { return "Favourite \(entry.fuel.displayName)" }
-        return entry.sort == .closest
+        switch entry.sort {
-            ? "Closest \(entry.fuel.displayName)"
+        case .closest: return "Closest \(entry.fuel.displayName)"
-            : "Cheapest \(entry.fuel.displayName)"
+        case .worthTheDrive: return "Best Value \(entry.fuel.displayName)"
        case .cheapest: return "Cheapest \(entry.fuel.displayName)"
        }
    }
    private var singleStation: some View {
@@ -85,7 +88,15 @@ struct FuelPriceWidgetContent: View {
            Text(station.name)
                .font(.headline)
                .lineLimit(1)
-            if let price = station.prices[entry.fuel] {
+            if entry.sort == .worthTheDrive, let result = entry.bestValueResult {
                Text(result.headline(compact: true))
                    .font(.title3.weight(.bold))
                    .lineLimit(1)
                Text(result.status == .baseline ? "your baseline" : "vs closest station")
                    .font(.caption2)
                    .foregroundStyle(.secondary)
                    .lineLimit(1)
            } else if let price = station.prices[entry.fuel] {
                FuelStore.priceTextAttributed(price, size: 26, weight: .bold, color: .green)
            }
            if let location = entry.location {
@@ -143,6 +143,13 @@ enum FuelSavingsFormat {
            : "\(distanceText) extra return travel"
    }
    /// Compact comparison copy kept outside the coloured verdict capsule.
    /// "Closest" refers to the existing shortest-journey baseline selection;
    /// it does not claim a straight-line comparison.
    static func comparisonLabel(status: FuelSavingsStatus) -> String {
        status == .baseline ? "your baseline" : "vs closest station"
    }
    /// VoiceOver form of `extraTravelLabel`. The abbreviation and the unit suffix
    /// are spoken as words ("Approximately 6.4 miles extra return travel") because
    /// synthesizers render "Approx." and "mi" inconsistently.
@@ -215,13 +222,11 @@ extension FuelSavingsResult {
        headline().replacingOccurrences(of: "Approx. ", with: "Approximately ")
    }
-    /// Row capsule copy: the headline, plus what the baseline row IS. Without the
+    /// Row capsule copy. The baseline relationship is shown as secondary text
-    /// suffix the baseline capsule just repeated the status word above it
+    /// beside the capsule in the station listing and detail summary, so the
-    /// ("Closest station" twice) and said nothing about the saving being measured
+    /// capsule itself contains only the verdict.
    /// against it.
    func capsuleHeadline() -> String {
-        let headline = headline(compact: true)
+        headline(compact: true)
        return status == .baseline ? "\(headline) · your baseline" : headline
    }
 }
@@ -183,13 +183,13 @@ enum DistanceUnit: String, Codable, CaseIterable, Identifiable {
    /// shown in this unit, rounded to a whole number for picker labels:
    /// 5 miles -> "5" (miles unit) or "8" (km unit).
    func displayMiles(_ miles: Int) -> Int {
-        Int(fromKM(Double(miles) * 1.60934).rounded())
+        Int(fromKM(Double(miles) * FuelSavingsCalculator.kilometresPerMile).rounded())
    }
    /// Convert a value expressed in this unit to km.
    func toKM(_ value: Double) -> Double {
        switch self {
-        case .miles: return value * 1.60934
+        case .miles: return value * FuelSavingsCalculator.kilometresPerMile
        case .kilometers: return value
        }
    }
@@ -197,7 +197,7 @@ enum DistanceUnit: String, Codable, CaseIterable, Identifiable {
    /// Convert a km value to this unit.
    func fromKM(_ km: Double) -> Double {
        switch self {
-        case .miles: return km * 0.621371
+        case .miles: return km / FuelSavingsCalculator.kilometresPerMile
        case .kilometers: return km
        }
    }
@@ -979,7 +979,7 @@ struct FuelStore {
    /// "Follow search" caps the geofence at 8 miles so a 10/15-mile Stations
    /// search never creates huge region-monitoring circles.
-    static let alertFollowCapKM: Double = 8 * 1.60934
+    static let alertFollowCapKM: Double = 8 * FuelSavingsCalculator.kilometresPerMile
    /// The effective alert radius: the manual radius, or — when alerts follow
    /// the Stations-tab search — that distance (miles, converted via the
@@ -1618,10 +1618,22 @@ struct FuelStore {
        // what the rows show.
        if accurateDistancesEnabled,
           accurateDistancesReady(userLat: userLat, userLng: userLng),
-           let meters = roadDistanceMeters(for: station, userLat: userLat, userLng: userLng) {
+           let entry = roadDistanceEntry(for: station, userLat: userLat, userLng: userLng) {
-            return StationDistanceDisplay(km: meters / 1000.0, isApproximate: false)
+            let returnKM = (entry.meters + (entry.returnMeters ?? entry.meters)) / 1000.0
            return StationDistanceDisplay(
                km: entry.meters / 1000.0,
                isApproximate: false,
                returnKM: returnKM,
                returnIsApproximate: entry.returnMeters == nil
            )
        }
-        return StationDistanceDisplay(km: station.distanceKM(to: userLat, lng2: userLng), isApproximate: true)
+        let km = station.distanceKM(to: userLat, lng2: userLng)
        return StationDistanceDisplay(
            km: km,
            isApproximate: true,
            returnKM: km * 2,
            returnIsApproximate: true
        )
    }
    /// Radius test on the SAME distance the UI shows. Filtering straight-line
@@ -1710,35 +1722,50 @@ struct RoadDistanceDiagnostics: Equatable {
 /// road distance or the straight-line fallback. The confidence travels WITH the
 /// value, so a label can never claim a confidence the figure doesn't have.
 struct StationDistanceDisplay: Equatable {
-    let km: Double
+let km: Double
-    let isApproximate: Bool
+let isApproximate: Bool
 let returnKM: Double
 let returnIsApproximate: Bool
-    /// Row/sheet label: a straight-line fallback is marked "Approx." so a figure
+init(km: Double,
-    /// that is NOT a routed road distance can never be read as one, and a cached
+     isApproximate: Bool,
-    /// road distance stays a bare figure. It updates by itself once routing lands,
+     returnKM: Double? = nil,
-    /// because `FuelStore.stationDistanceDisplay` re-reads the cache every call.
+     returnIsApproximate: Bool? = nil) {
-    func label(unit: DistanceUnit) -> String {
+    self.km = km
    self.isApproximate = isApproximate
    self.returnKM = returnKM ?? km * 2
    self.returnIsApproximate = returnIsApproximate ?? isApproximate
 }
 /// Row/sheet label: a straight-line fallback is marked "Approx." so a figure
 /// that is NOT a routed road distance can never be read as one, and a cached
 /// road distance stays a bare figure. It updates by itself once routing lands,
 /// because `FuelStore.stationDistanceDisplay` re-reads the cache every call.
 func label(unit: DistanceUnit) -> String {
    let text = unit.format(km)
    return isApproximate ? "Approx. \(text)" : text
-    }
+}
-    /// Row/sheet label that keeps the one-way station distance primary, then
+/// Row/sheet label that keeps the one-way station distance primary, then
-    /// adds the out-and-back distance in brackets. This matches Best Value's
+/// adds the independently measured or estimated out-and-back distance.
-    /// "extra return travel" basis without changing the actual distance model.
+func labelWithReturn(unit: DistanceUnit) -> String {
-    func labelWithReturn(unit: DistanceUnit) -> String {
+    let returnText = unit.format(returnKM)
-        "\(label(unit: unit)) (\(unit.format(km * 2)) return)"
+    let returnLabel = returnIsApproximate ? "Approx. \(returnText)" : returnText
-    }
+    return "\(label(unit: unit)) (\(returnLabel) return)"
 }
-    /// VoiceOver form of `label(unit:)`.
+/// VoiceOver form of `label(unit:)`.
-    func spokenLabel(unit: DistanceUnit) -> String {
+func spokenLabel(unit: DistanceUnit) -> String {
    let text = unit.spoken(km)
    return isApproximate ? "Approximately \(text)" : text
-    }
+}
-    /// VoiceOver form of `labelWithReturn(unit:)`.
+/// VoiceOver form of `labelWithReturn(unit:)`.
-    func spokenLabelWithReturn(unit: DistanceUnit) -> String {
+func spokenLabelWithReturn(unit: DistanceUnit) -> String {
-        "\(spokenLabel(unit: unit)), \(unit.spoken(km * 2)) return"
+    let returnText = unit.spoken(returnKM)
-    }
+    let returnLabel = returnIsApproximate ? "Approximately \(returnText)" : returnText
    return "\(spokenLabel(unit: unit)), \(returnLabel) return"
 }
 }
 /// Cached Apple-Maps road distances for nearby stations (see
@@ -21,7 +21,7 @@ enum SiriCheapestLookup {
        lng: Double,
        withinMiles: Double
    ) -> FuelStation? {
-        let radiusKM = withinMiles * 1.60934
+        let radiusKM = withinMiles * FuelSavingsCalculator.kilometresPerMile
        // Same radius rule as the app list: the distance the answer would SHOW
        // (cached road route where there is one).
        return stations
Author	SHA1	Message	Date
FuelBoard Contributor	39a97f4905	Standardise onboarding header sizes	2026-10-15 17:23:25 +01:00
FuelBoard Contributor	3819b8a6ab	Add Best Value small widget and standardise Closest wording	2026-10-15 17:14:02 +01:00
FuelBoard Contributor	a5688cbac2	Clarify fuel savings comparison labels	2026-10-15 11:38:24 +01:00
FuelBoard Contributor	558f956d8d	Update FuelBoard onboarding copy, title formatting, key sheet card wrapping, and best-value button placement	2026-10-14 16:22:31 +01:00
FuelBoard Contributor	f04a0fe0f5	Key sheet: gate the How Best Value row on the feature, like the More info row The More info row went inside the Best Value card's `if let fuelSavings`, but the Key sheet row was left ungated, so it stayed visible with Best Value switched off. It is now gated on `fuelSavingsSettings.isConfigured` — exactly the condition `savingsForStation` uses, i.e. the same gate as the card — so the two rows appear and disappear together. Verified on the simulator by toggling Best Value in Settings and scrolling the Key sheet to its end (four drags, sheet still up and its footer in view each time): - OFF: the row is absent at every scroll position, and the More info sheet shows neither the card nor the row; - ON: the row is present, and the More info card and its row are both back. 210 tests pass; Release builds.	2026-10-14 15:19:20 +01:00
FuelBoard Contributor	f35f605b58	Onboarding welcome: the app icon stands on its own, with no circle behind it The first onboarding screen showed the app icon inside AnimatedOnboardingIcon's soft circle — a 10% tinted disc with an 18% ring around it. The wrapper now takes `showsHalo`, defaulting to true so the other five pages and BestValueIntroView keep theirs unchanged; the welcome page passes false. Verified with before/after installs on the simulator: - the icon's artwork is pixel-identical and unmoved (centre 603,661, 95 pt); - the halo is gone — the tile's non-background footprint falls from the 118 pt ring to exactly the 96 pt tile, and the ring pixels at r=59 pt lose their tint; - page 1 differs only by the status-bar clock, so the other pages are untouched. The copy below the icon tightens by ~11 pt: the page block is centred between two spacers and the icon's container no longer contributes 22 pt of height. The page dots and the Continue button do not move. The icon keeps its soft accent shadow.	2026-10-14 15:02:29 +01:00
FuelBoard Contributor	7da277547b	More info sheet: the copy confirmation reads "Postcode copied" The capsule label was just "Copied", which reads oddly above a row whose hint is "Copies the postcode". It is now "Postcode copied". Still centred over the address row (centre x 603 of a 1206 px screen) and still a pure overlay: the bands above and below the capsule are pixel-identical while it is up (no reflow), and the row stays tappable straight through it. The "Copied" localisation key goes with its only use; "Postcode copied" replaces it.	2026-10-14 14:52:07 +01:00
FuelBoard Contributor	93df94ddc9	How Best Value works: its own sheet, opened from the Key sheet and More info sheet The explanation used to be a section inside the Key sheet. It is a standalone HowBestValueSheet now, opened from a tappable row with a chevron: at the bottom of the Key sheet, and at the bottom of the More info sheet below the Best Value card. Gating: the More info row sits INSIDE the same `if let fuelSavings` as the card it follows, so it disappears with the feature. Verified with Best Value toggled off in Settings: the More info sheet then shows neither the card nor the row. The Key sheet row is deliberately ungated — that sheet is the legend and is reachable whatever the feature is doing. Copy is unchanged: the same Localizable.strings strings are reused, and the old section header became the sheet's title. A DEBUG-only `-showBestValueInfo` hook opens the sheet without a tap.	2026-10-14 14:37:18 +01:00
FuelBoard Contributor	78b43ce6d7	Merge branch 'feature/more-info-map-and-postcode-copy' into integration/more-info-and-best-value	2026-10-14 13:57:21 +01:00
FuelBoard Contributor	7dbb3d1a15	More info sheet: float the "Copied" capsule over the address row, not the map The confirmation belongs to the address row, so hovering it over the map read as belonging to the map. It now overlays the address row itself, horizontally centred on the address line, as .overlay(alignment: .center) on the row with allowsHitTesting(false) still keeping the row tappable through the capsule. Verified: with the capsule up, the OCR box sits at y 1189-1231 inside the address band (map bottom is 1075), the tint blob is centred at x 602.5 against a screen centre of 603, the row's ink runs outside the capsule footprint are byte-identical to the idle frame, and a real tap on the capsule's own footprint still copies.	2026-10-14 13:57:19 +01:00
FuelBoard Contributor	13967c3782	Merge branch 'feature/more-info-map-and-postcode-copy' into integration/more-info-and-best-value	2026-10-14 13:45:05 +01:00
FuelBoard Contributor	6e534ed5b7	More info sheet: float the "Copied" capsule and put the copy glyph on the address line The address row is back to a single line: the copy glyph now sits on the address line itself (trailing, plain glyph — the confirmation says it elsewhere), and the always-reserved second line is gone. The "Copied" confirmation is now a green-tinted capsule that floats over the bottom of the map, directly above the address and horizontally centred. It is an overlay with allowsHitTesting(false), so nothing reflows while it is up (verified: the address row's ink, column for column, does not move) and the map's tap target stays live underneath (verified: tapping where the capsule floats still opens Apple Maps). A material base keeps it legible against map tiles, where a 14% wash alone — the notice-capsule recipe — disappears. Timing is unchanged: 2s, then fades, with the success haptic on copy.	2026-10-14 13:45:05 +01:00
FuelBoard Contributor	4718e9b76a	Merge branch 'feature/more-info-map-and-postcode-copy' into integration/more-info-and-best-value	2026-10-14 13:16:20 +01:00
FuelBoard Contributor	73db893f4f	More info sheet: drop the hairline between the map and the address The map already reads as its own block, so the row separator below it only added clutter. Removed with .listRowSeparator(.hidden, edges: .bottom) on the map row, leaving the Address/Distance and Distance/actions hairlines in place.	2026-10-14 13:16:19 +01:00
FuelBoard Contributor	6dfe732199	Merge branch 'feature/more-info-map-and-postcode-copy' into integration/more-info-and-best-value	2026-10-14 12:48:33 +01:00
FuelBoard Contributor	7da09928ca	More info sheet: one card for the map, address, distance and the two actions The map, the address, the distance, any notices and the Directions/Report issue row now share a single section, in that order, with the buttons as the last row. The list's own row hairlines separate them (inset like Address/Distance), so no hand-rolled Divider is added — that would double the line. The map loses .listRowInsets(EdgeInsets()) so it sits inset inside the card with its rounded corners, aligned with the hairlines and text rather than butting the card edges. Directions/Report issue keep .buttonStyle(.borderless); verified with real simulated taps in the merged card that each fires alone.	2026-10-14 12:48:33 +01:00
FuelBoard Contributor	f0c13a79d6	Merge branch 'feature/more-info-map-and-postcode-copy' into integration/more-info-and-best-value	2026-10-14 12:25:17 +01:00
FuelBoard Contributor	666d59c483	More info sheet: give each action button its own tap target Directions and Report issue both fired on any tap inside their row: with the default button style a List row has a single row-wide tap target, so tapping one button triggered the other's action too (tapping Report issue opened Apple Maps, and tapping Directions showed the report alert). `.buttonStyle(.borderless)` gives each button its own hit area over its half-row frame. Verified with real simulated taps on both buttons.	2026-10-14 12:25:16 +01:00
FuelBoard Contributor	311118f0fa	Add a DEBUG screen-marker hook so Simulator taps can be verified `-qaScreenMarker` paints the screen flat magenta, which lets a macOS capture locate the device-screen rectangle exactly and map simulator-screenshot pixels to click coordinates. `simctl` cannot drive taps and the project has no UI-test target, so this is the only way to test real touch behaviour on the sheet. Guarded by #if DEBUG, exactly like the existing -qaCopyPostcode hook.	2026-10-14 12:25:16 +01:00
FuelBoard Contributor	6b41e8ac51	Merge branch 'feature/more-info-map-and-postcode-copy' into integration/more-info-and-best-value	2026-10-14 12:07:54 +01:00
FuelBoard Contributor	1b2bdd46ea	More info sheet: give the two station actions their own card below the map Sharing the map's section made the buttons read as part of the map card and the map tap target. They now sit in their own section, rendering as a separate card directly beneath the map.	2026-10-14 12:07:54 +01:00
FuelBoard Contributor	e58dbb7833	Merge branch 'feature/more-info-map-and-postcode-copy' into integration/more-info-and-best-value	2026-10-14 11:52:42 +01:00
FuelBoard Contributor	6983fa83cc	More info sheet: put Directions and Report issue on one line under the map The two station actions now sit together directly beneath the map card, each taking half the width, instead of being two separate rows at the foot of the sheet.	2026-10-14 11:52:41 +01:00
FuelBoard Contributor	7ee1f06ce6	Merge branch 'feature/best-value-candidate-fill' into integration/more-info-and-best-value	2026-10-14 11:34:33 +01:00
FuelBoard Contributor	68139e6690	More info sheet: overlay the map tap target, move the copy line below the address The map tap target is now an OVERLAY button on top of the Map. A Map wrapped in a Button inside a List row never received the touch, so the first attempt left the card inert. The Map stays non-interactive and the overlay owns an explicit rounded-rect content shape. The copy glyph and its Copied confirmation move onto their own always-present second line, so the confirmation can no longer reflow the address above it. Verified: the address line's ink-pixel runs are identical with and without the confirmation.	2026-10-14 11:26:21 +01:00
FuelBoard ContributorandFuelBoard Contributor	d537667aa3	More info sheet: tappable map opens directions, address copies postcode	2026-10-14 11:26:08 +01:00