From 1f4d6a28a546ce9ed34a87221b352610396242f2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= <mbombeck@gmail.com>
Date: Sun, 10 May 2026 13:43:04 +0200
Subject: [PATCH] fix(settings): unify threshold page naming
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The thresholds settings route was carrying three different
names: nav label "Personal targets", page header "Personal
targets", inner card header "Personal thresholds". The URL
slug is `/settings/thresholds`, so settled on "Personal
thresholds" everywhere in EN; DE keeps the established
"Persönliche Zielwerte". While here, dropped the duplicate
title + subtitle inside the editor card — the page-level
header already provides them and the second copy was just
visual noise.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 messages/en.json                                   |  2 +-
 .../settings/thresholds-editor-section.tsx         | 14 +++++++++-----
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/messages/en.json b/messages/en.json
index 4facece9f..b8cfaa382 100644
--- a/messages/en.json
+++ b/messages/en.json
@@ -1064,7 +1064,7 @@
         "description": "Tile layout and order."
       },
       "thresholds": {
-        "title": "Personal targets",
+        "title": "Personal thresholds",
         "description": "Override target ranges for every metric."
       },
       "ai": {
diff --git a/src/components/settings/thresholds-editor-section.tsx b/src/components/settings/thresholds-editor-section.tsx
index b2a09558a..37879893d 100644
--- a/src/components/settings/thresholds-editor-section.tsx
+++ b/src/components/settings/thresholds-editor-section.tsx
@@ -123,10 +123,17 @@ export function ThresholdsEditorSection({ id }: { id: string }) {
       id={id}
       className="bg-card border-border scroll-mt-28 space-y-5 rounded-xl border p-6"
     >
+      {/* v1.4.19 A8 / F-07: page header `settings.sections.thresholds.*`
+          already provides the title + description for this route, so the
+          card-level title + subtitle were a duplicate of the page header.
+          Keep the icon + reset action so the card still has a visible
+          control affordance. */}
       <div className="flex items-center justify-between">
         <div className="flex items-center gap-2">
-          <SlidersHorizontal className="text-primary h-5 w-5" />
-          <h2 className="text-lg font-semibold">{t("thresholds.title")}</h2>
+          <SlidersHorizontal
+            className="text-primary h-5 w-5"
+            aria-hidden="true"
+          />
         </div>
         {data && Object.keys(data.overrides).length > 0 && (
           <Button
@@ -140,9 +147,6 @@ export function ThresholdsEditorSection({ id }: { id: string }) {
           </Button>
         )}
       </div>
-      <p className="text-muted-foreground text-sm">
-        {t("thresholds.subtitle")}
-      </p>
 
       {isLoading || !data ? (
         <div className="text-muted-foreground flex items-center gap-2 text-sm">