From 4936aa78caf7adf15716744b8d3412bbce903469 Mon Sep 17 00:00:00 2001 From: r-zakarya <82443831+r-zakarya@users.noreply.github.com> Date: Sat, 1 Aug 2026 12:47:59 +0100 Subject: [PATCH] refactor(entries): drop the cyan secret marker on the reveal eye The eye only renders on a secret field, so its presence already carries the state. --- css/style.css | 9 ++------- js/app.js | 7 +++---- 2 files changed, 5 insertions(+), 11 deletions(-) diff --git a/css/style.css b/css/style.css index 3368aee..976e9eb 100644 --- a/css/style.css +++ b/css/style.css @@ -1675,13 +1675,8 @@ input[type="range"]::-webkit-slider-thumb { .so-combo-item:hover { background: var(--accent); color: #fff; } /* The eye carries the secret state itself — a second toggle beside it read as a duplicate of the reveal. Flipping the flag lives in the row menu. */ -/* Cyan marker = the field is STORED as a secret, not merely hidden right now. */ -.so-custom-row .icon-btn.is-secret { - color: var(--accent); - background: var(--accent-soft); - border: 1px solid var(--accent); - border-radius: 4px; -} +/* No marker for "stored as secret": the eye only exists on a secret field — + Make public removes it — so its mere presence already says so. */ .so-custom-add { margin-top: 8px; } diff --git a/js/app.js b/js/app.js index f0596d0..2baa13e 100644 --- a/js/app.js +++ b/js/app.js @@ -4277,11 +4277,10 @@ function buildCustomFieldRow(field, idx, rerender) { }); // The secret flag no longer has a button of its own: two adjacent toggles - // that both masked the value read as duplicates. The eye alone carries the - // state (cyan ring = stored as secret) and flipping it lives in the row - // menu, one deliberate step away from the everyday reveal. + // that both masked the value read as duplicates. The eye is shown only on a + // secret field, so its presence IS the indicator, and flipping the flag + // lives in the row menu, one deliberate step away from the everyday reveal. function applySecretState() { - eye.classList.toggle('is-secret', !!field.is_secret); eye.style.visibility = (!field.is_secret || (valueInput.tagName !== 'INPUT' && !field.multiline)) ? 'hidden' : '';