From fa8f7e2418c7f84fd6952020e64b94a642ac52f6 Mon Sep 17 00:00:00 2001
From: Guillem Arias <accounts@gariasf.com>
Date: Mon, 11 May 2026 15:29:05 +0200
Subject: [PATCH] fix(savings_goals/new): stepper line uses border-secondary
 token
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

bg-secondary is not a registered Sure utility, so the previous h-0.5
bg-secondary div rendered with no background — the line was just a
transparent slot, barely visible in either theme.

Replace with border-t-2 border-secondary on the connector div, and
toggle border-inverse / border-secondary on step transition. Both
classes are real Sure tokens with proper light/dark variants
(alpha-black-200 / alpha-white-300 for border-secondary).
---
 app/javascript/controllers/savings_goal_stepper_controller.js | 4 ++--
 app/views/savings_goals/_form_stepper.html.erb                | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/javascript/controllers/savings_goal_stepper_controller.js b/app/javascript/controllers/savings_goal_stepper_controller.js
index 72753ba43..9c5c4b5f3 100644
--- a/app/javascript/controllers/savings_goal_stepper_controller.js
+++ b/app/javascript/controllers/savings_goal_stepper_controller.js
@@ -194,8 +194,8 @@ export default class extends Controller {
       this.step2CircleTarget.classList.toggle("text-secondary", this.currentStep < 2);
     }
     if (this.hasStepperLineTarget) {
-      this.stepperLineTarget.classList.toggle("bg-inverse", this.currentStep > 1);
-      this.stepperLineTarget.classList.toggle("bg-secondary", this.currentStep === 1);
+      this.stepperLineTarget.classList.toggle("border-inverse", this.currentStep > 1);
+      this.stepperLineTarget.classList.toggle("border-secondary", this.currentStep === 1);
     }
     // Modal subtitle lives in the dialog header, outside this controller's
     // DOM scope. Locate it by attribute and update directly.
diff --git a/app/views/savings_goals/_form_stepper.html.erb b/app/views/savings_goals/_form_stepper.html.erb
index 43451be8e..9227d63db 100644
--- a/app/views/savings_goals/_form_stepper.html.erb
+++ b/app/views/savings_goals/_form_stepper.html.erb
@@ -11,7 +11,7 @@
       <span data-savings-goal-stepper-target="step1Circle" class="w-7 h-7 rounded-full inline-flex items-center justify-center bg-inverse text-inverse text-xs font-medium">1</span>
       <span class="text-sm font-medium text-primary"><%= t("savings_goals.form_stepper.step1.label") %></span>
     </div>
-    <div class="flex-1 h-0.5 bg-secondary rounded-full" data-savings-goal-stepper-target="stepperLine"></div>
+    <div class="flex-1 border-t-2 border-secondary" data-savings-goal-stepper-target="stepperLine"></div>
     <div class="flex items-center gap-2" data-savings-goal-stepper-target="step2Indicator">
       <span data-savings-goal-stepper-target="step2Circle" class="w-7 h-7 rounded-full inline-flex items-center justify-center border border-secondary text-secondary text-xs font-medium">2</span>
       <span class="text-sm font-medium text-secondary"><%= t("savings_goals.form_stepper.step2.label") %></span>