From 3e1a14f3348ba04ccac8e2a0bf12865e44972b7e Mon Sep 17 00:00:00 2001
From: Giorgio Gilestro <giorgio@gilest.ro>
Date: Fri, 29 May 2026 11:23:52 +0200
Subject: [PATCH] =?UTF-8?q?ui:=20flip=20tone=20relabel=20=E2=80=94=20"Pro"?=
 =?UTF-8?q?=20now=20maps=20to=20INTERMEDIATE,=20not=20NOVICE?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Reverses the polarity of 71155a6 to match the actual semantics:

- "Novice" stays labelled "Novice" → glossary tooltips, plainer prose.
- "Intermediate" is relabelled "Pro" → terse, assumes fluency, no
  hand-holding. This is the mode an expert reader wants, so the "Pro"
  badge actually fits.

Backend tone values (NOVICE, INTERMEDIATE) are unchanged — no API,
prompt, or stored-preference impact. Only the display strings flip.

Also drops the .tone-toggle button min-width: 10em override added in
71155a6. With "Intermediate" gone from the visible label, the longest
remaining label is "Novice" (6 chars), which fits the shared 5.5em
just like the theme and language toggles.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
---
 app/static/css/layout.css   |  5 -----
 app/templates/base.html     | 13 +++++++------
 app/templates/settings.html |  4 ++--
 3 files changed, 9 insertions(+), 13 deletions(-)

diff --git a/app/static/css/layout.css b/app/static/css/layout.css
index fc94c65..83f03f5 100644
--- a/app/static/css/layout.css
+++ b/app/static/css/layout.css
@@ -157,11 +157,6 @@ body.drawer-open .drawer-backdrop { opacity: 1; }
 .tone-toggle button + button,
 .theme-toggle button + button,
 .lang-toggle button + button { border-left: 1px solid var(--border); }
-/* The tone-toggle's longer option ("Intermediate", 12 chars) needs more
-   room than the shared 5.5em min-width. We size both buttons to fit the
-   longest one so the popup width (set by container width via left/right:0)
-   doesn't get clipped when only the short "Pro" label is active. */
-.tone-toggle button { min-width: 10em; }
 .tone-toggle button:hover,
 .theme-toggle button:hover,
 .lang-toggle button:hover { color: var(--accent); }
diff --git a/app/templates/base.html b/app/templates/base.html
index 7344b76..97028ab 100644
--- a/app/templates/base.html
+++ b/app/templates/base.html
@@ -234,16 +234,17 @@
         <a href="/log"  class="{% if request.url.path.startswith('/log') %}active{% endif %}">Log</a>
       </nav>
       <div class="header-right">
-        {# The "Pro" label maps to the NOVICE tone server-side — kept that
-           way to avoid touching every stored user preference and API
-           contract. The mode itself (glossary tooltips + plainer
-           framing) is unchanged; only the display label changes. #}
+        {# The "Pro" label maps to the INTERMEDIATE tone server-side —
+           kept that way to avoid touching every stored user preference
+           and API contract. The mode itself (terse, no glossary
+           tooltips, assumes fluency) is unchanged; only the display
+           label changes. #}
         <div id="tone-toggle" class="tone-toggle" data-tone="INTERMEDIATE"
              role="group" aria-label="Explanation level">
           <button type="button" data-value="NOVICE"
-                  onclick="cassandraSetTone('NOVICE')">Pro</button>
+                  onclick="cassandraSetTone('NOVICE')">Novice</button>
           <button type="button" data-value="INTERMEDIATE"
-                  onclick="cassandraSetTone('INTERMEDIATE')">Intermediate</button>
+                  onclick="cassandraSetTone('INTERMEDIATE')">Pro</button>
         </div>
         <div id="theme-toggle" class="theme-toggle" data-theme="light"
              role="group" aria-label="Theme">
diff --git a/app/templates/settings.html b/app/templates/settings.html
index 37e88f7..ac0107d 100644
--- a/app/templates/settings.html
+++ b/app/templates/settings.html
@@ -185,9 +185,9 @@
         <div class="settings-row__value">
           <div style="display:flex; gap:14px;">
             <label><input type="radio" name="digest-tone" value="NOVICE"
-              {% if (user.digest_tone or 'INTERMEDIATE') == 'NOVICE' %}checked{% endif %}> Pro</label>
+              {% if (user.digest_tone or 'INTERMEDIATE') == 'NOVICE' %}checked{% endif %}> Novice</label>
             <label><input type="radio" name="digest-tone" value="INTERMEDIATE"
-              {% if (user.digest_tone or 'INTERMEDIATE') == 'INTERMEDIATE' %}checked{% endif %}> Intermediate</label>
+              {% if (user.digest_tone or 'INTERMEDIATE') == 'INTERMEDIATE' %}checked{% endif %}> Pro</label>
           </div>
         </div>
       </div>