From f261695b86d89d799810bb710610612a81b8a1e0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=D0=90=D0=BD=D1=82=D0=BE=D0=BD=20=D0=A2=D1=80=D0=BE=D1=88?=
 =?UTF-8?q?=D0=B8=D0=BD?= <i@atroshin.ru>
Date: Thu, 1 Oct 2026 21:57:15 +0300
Subject: [PATCH] ui: the keyset row's remove button uses the red action style
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The ✕ in the Cards keyset editor was a neutral bordered button with only a
red hover; it now matches the preset list's Remove (bg-red-600 text-white,
hover bg-red-700), keeping the glyph and the "Remove keyset" tooltip.

keysets.test.js guards the red style.  sw cache simple-v345.
---
 frontend/index.html            | 2 +-
 frontend/sw.js                 | 2 +-
 frontend/tests/keysets.test.js | 2 ++
 3 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/frontend/index.html b/frontend/index.html
index 77fdefd..898debe 100644
--- a/frontend/index.html
+++ b/frontend/index.html
@@ -11404,7 +11404,7 @@ function cardsKeysetRowHtml(ks, i) {
 		'<input id="cards-ks-' + i + '-kic-key" class="flex-1 ' + cls + '" placeholder="KIc key (hex)" value="' + esc(ks.kicKey || '') + '">' +
 		'<input id="cards-ks-' + i + '-kid-key" class="flex-1 ' + cls + '" placeholder="KID key (hex)" value="' + esc(ks.kidKey || '') + '">' +
 		'<input id="cards-ks-' + i + '-cntr" class="w-24 ' + cls + '" maxlength="10" placeholder="Counter" value="' + esc(ks.cntr || '') + '">' +
-		'<button type="button" onclick="cardsKeysetRemove(' + i + ')" class="px-2 py-1 text-xs rounded border border-gray-300 dark:border-slate-600 hover:bg-red-100 dark:hover:bg-red-900/40" title="' + esc(t('Remove keyset')) + '">\u2715</button>' +
+		'<button type="button" onclick="cardsKeysetRemove(' + i + ')" class="px-2 py-1 text-xs rounded bg-red-600 text-white hover:bg-red-700" title="' + esc(t('Remove keyset')) + '">\u2715</button>' +
 		'</div>';
 }
 
diff --git a/frontend/sw.js b/frontend/sw.js
index 6930229..8c39226 100644
--- a/frontend/sw.js
+++ b/frontend/sw.js
@@ -1,4 +1,4 @@
-const CACHE = 'simple-v344';
+const CACHE = 'simple-v345';
 const URLS = [
   'index.html',
   'help.html',
diff --git a/frontend/tests/keysets.test.js b/frontend/tests/keysets.test.js
index 703051d..37ccb01 100644
--- a/frontend/tests/keysets.test.js
+++ b/frontend/tests/keysets.test.js
@@ -111,6 +111,8 @@ test('the Cards form wires the keyset editor', () => {
 	assert.match(html, /onclick="cardsKeysetAdd\(\)"/);
 	assert.match(html, /oninput="cardsKeysetKvnUpdate\(/);
 	assert.match(html, /data-l10n="Add keyset"/);
+	// the remove button is the red action style, like the preset list's Remove
+	assert.match(html, /cardsKeysetRemove\(' \+ i \+ '\)" class="[^"]*bg-red-600 text-white/);
 	// the keyset rows are built with the KIc/KID/key/counter inputs
 	assert.match(html, /cards-ks-' \+ i \+ '-kic-key/);
 	assert.match(html, /cards-ks-' \+ i \+ '-cntr/);