From 03ead1722f4c57616adc6c709ebbce0642c03cfb Mon Sep 17 00:00:00 2001
From: Lukasz Kasprzak <lukas@labunix.xyz>
Date: Wed, 29 Jul 2026 20:46:57 +0200
Subject: web(settings): drop the terminal "versions" field; it is CLI-only

The settings page showed two near-identical version checkbox rows --
"versions" (the terminal compare set) and "web_versions" (the web page's
own default ticks). The first only affects the CLI, so editing it from
the web is pointless and was actively confusing (both rows list the same
wuj/vul/grb/drb boxes).

Remove the "versions" row from the form and stop reading it in
settingsPost, so cfg.Versions is preserved from the live config (same
pattern as offline/width/pager). The web settings now expose only what
the web uses: default_version and web_versions.
---
 internal/web/templates/settings.html | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

(limited to 'internal/web/templates/settings.html')

diff --git a/internal/web/templates/settings.html b/internal/web/templates/settings.html
index 8e18278..cb753ae 100644
--- a/internal/web/templates/settings.html
+++ b/internal/web/templates/settings.html
@@ -67,11 +67,10 @@
       </select>
     </label>
 
-    <div class="row">
-      {{.L.WebVersions}}
-      {{range .VersionOpts}}<label><input type="checkbox" name="versions" value="{{.Code}}" {{if .Checked}}checked{{end}}> {{.Code}}</label>{{end}}
-    </div>
-
+    {{/* The terminal compare set (config "versions") is intentionally NOT here:
+         it only affects the CLI, so editing it from the web is pointless and
+         confusing. settingsPost preserves it. This page keeps only the web's
+         own defaults: the single default_version and the web_versions ticks. */}}
     <div class="row">
       {{.L.WebWebVersions}}
       {{range .WebVersionOpts}}<label><input type="checkbox" name="web_versions" value="{{.Code}}" {{if .Checked}}checked{{end}}> {{.Code}}</label>{{end}}
-- 
cgit v1.3