Fix invisible service descriptions, broken empty-translation glitch; redesign modal list
Real bugs, both confirmed via screenshot 2026-08-02: - dark3 was set to a near-white cream tone assuming background-only use, but Klaro's .cm-list-description rule uses it as a TEXT color — every service description was almost invisible. Changed to --color-text-muted. - The service.disableAll.description="" override to hide Klaro's boilerplate toggle-all text broke its own t() lookup, rendering a literal "[missing translation: de/service/disableAll/description]" string instead — worse than the original. Reverted, hidden via a plain CSS display:none on .cm-toggle-all .cm-list-description instead. Also a real redesign pass on the settings modal's service/purpose list (wider modal, row separators, bigger/clearer toggle switches, distinct footer) — was previously only reachable via the notice's own generic styling, still visibly "default library layout" per Marco's own read. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
@@ -115,7 +115,13 @@ export function buildKlaroConfig(codes: TrackingCode[], onAccept: (code: Trackin
|
||||
theme: ["light", "bottom", "left"],
|
||||
dark1: "#fffdf8", // --color-bg-paper — notice/modal background
|
||||
dark2: "#e5e0d8", // --color-border
|
||||
dark3: "#f3efe9",
|
||||
// NOT a light background tint despite the name — Klaro's own
|
||||
// .cm-list-description rule (each service's description text in
|
||||
// the modal) uses dark3 as a TEXT color, not a background. A near-
|
||||
// white value here (tried first) made every service description
|
||||
// almost invisible against the modal's own light background —
|
||||
// confirmed via screenshot 2026-08-02. --color-text-muted instead.
|
||||
dark3: "#6b6b69",
|
||||
light1: "#1a1a18", // --color-text-primary — notice/modal text
|
||||
light2: "#6b6b69", // --color-text-muted
|
||||
light3: "#1a1a18",
|
||||
@@ -155,13 +161,6 @@ export function buildKlaroConfig(codes: TrackingCode[], onAccept: (code: Trackin
|
||||
marketing: "Marketing",
|
||||
content: "Inhalte",
|
||||
},
|
||||
// Klaro's own bundled German copy for the per-service "alle
|
||||
// umschalten" toggle in the modal — the explanatory line reads as
|
||||
// redundant boilerplate next to a plainly-labeled switch, and the
|
||||
// toggle's own text color made it barely legible anyway (see
|
||||
// KlaroTheme's own comment). Title stays (Klaro's own default is
|
||||
// fine), only the description is emptied.
|
||||
service: { disableAll: { description: "" } },
|
||||
...serviceTranslations,
|
||||
},
|
||||
},
|
||||
|
||||
Reference in New Issue
Block a user