From 7ec34e339b045d1fe4a23746a897602aa30cc8b1 Mon Sep 17 00:00:00 2001 From: NguyenND Date: Wed, 19 Aug 2026 12:13:50 -0400 Subject: [PATCH] Aug 19 - Update: remove theme switch option --- CLAUDE.md | 4 +++- app/templates/layouts/modern.html | 24 +++++------------------- 2 files changed, 8 insertions(+), 20 deletions(-) diff --git a/CLAUDE.md b/CLAUDE.md index 3a9326f..020424c 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -1047,7 +1047,9 @@ sudo systemctl restart gunicorn Revision id `phase50_default_modern`. Promotes the phase48 A/B-test design to the default. Two changes, **both required**: the `users.ui_theme` column default becomes `'modern'` (new accounts), and existing rows are moved `'classic'` → `'modern'`. phase48 stored a literal `'classic'` for everyone rather than NULL, so a default change alone would leave every current user on the old design. -**It overwrites a deliberate choice** — phase48 gave no way to tell "I picked classic" from "I never touched it", so anyone who actively preferred classic is moved too. They can switch back from the account menu and that choice then sticks; the switcher is unchanged. `/ui/theme-votes` reads the same column, so the tally reads 100% modern afterwards — capture it first if the numbers matter. `downgrade()` returns *everyone* to classic (individual prior choices were never recorded). +**It overwrites a deliberate choice** — phase48 gave no way to tell "I picked classic" from "I never touched it", so anyone who actively preferred classic is moved too. + +**The A/B test is closed (Aug 2026).** Modern is THE design. The **"Back to Classic Design"** switcher and the admin **"Design Vote Tally"** link were removed from the modern account menu, so a user on modern stays there. Two things were deliberately KEPT: the classic layout still offers **"Try the New Design"** (the only way off classic for anyone still on it — removing it would strand them on a retired design), and `ui.switch_theme` / `ui.theme_votes` still work if visited directly, so nothing 404s mid-request and the tally is still readable at `/ui/theme-votes`. Nobody's stored `ui_theme` was changed by this. `/ui/theme-votes` reads the same column, so the tally reads 100% modern afterwards — capture it first if the numbers matter. `downgrade()` returns *everyone* to classic (individual prior choices were never recorded). To change the default without touching saved preferences, set `DEFAULT_UI_THEME=classic` instead of running this. diff --git a/app/templates/layouts/modern.html b/app/templates/layouts/modern.html index ea5f6be..10f2b01 100644 --- a/app/templates/layouts/modern.html +++ b/app/templates/layouts/modern.html @@ -164,25 +164,11 @@ Notification Preferences -
  • -
  • - {# ── Design switcher (modern → classic) ── #} -
    - - - - -
    -
  • - {% if current_user.role == 'admin' %} -
  • - - Design Vote Tally - -
  • - {% endif %} + {# The design A/B test is over — modern is THE design (Aug 2026). + The switcher and the vote tally are gone from this menu. + ui.switch_theme / ui.theme_votes still exist and still work + if visited directly, so nothing is stranded mid-request; + they are simply no longer offered. #}