From d49a1f68373594334c107f790416b2b7af258be2 Mon Sep 17 00:00:00 2001
From: Dmitriy Petrov <floplopru@gmail.com>
Date: Sun, 12 Jul 2026 00:14:54 +0300
Subject: [PATCH] Raise delete strike overlay above row content

---
 src/shared/versions.js                    |  4 ++--
 src/web/components/ClientOverviewPage.jsx | 10 +++++-----
 src/web/styles.css                        | 12 +++++++++---
 test/web/rule-editor-contract.test.js     |  3 ++-
 4 files changed, 18 insertions(+), 11 deletions(-)

diff --git a/src/shared/versions.js b/src/shared/versions.js
index d39f15d..35fd898 100644
--- a/src/shared/versions.js
+++ b/src/shared/versions.js
@@ -1,6 +1,6 @@
 export const HARBOR_VERSIONS = Object.freeze({
-  macClient: '0.6.7',
-  gatewayClient: '0.6.6',
+  macClient: '0.6.9',
+  gatewayClient: '0.6.8',
   gatewayBackend: '0.6.0',
 });
 
diff --git a/src/web/components/ClientOverviewPage.jsx b/src/web/components/ClientOverviewPage.jsx
index cd07b51..612c228 100644
--- a/src/web/components/ClientOverviewPage.jsx
+++ b/src/web/components/ClientOverviewPage.jsx
@@ -392,11 +392,6 @@ function LocalRulesPanel({
                   style={{ viewTransitionName: rule.removing ? 'none' : rule._key }}
                   inert={rule.removing ? true : undefined}
                 >
-                  <span
-                    className="client-delete-strike"
-                    aria-hidden="true"
-                    onAnimationEnd={() => onRemoveComplete(rule._key)}
-                  />
                   <button
                     className="client-local-rule-enabled"
                     type="button"
@@ -436,6 +431,11 @@ function LocalRulesPanel({
                   >
                     ×
                   </button>
+                  <span
+                    className="client-delete-strike"
+                    aria-hidden="true"
+                    onAnimationEnd={() => onRemoveComplete(rule._key)}
+                  />
                 </div>
                 );
               })}
diff --git a/src/web/styles.css b/src/web/styles.css
index 93b8d78..eaf6b63 100644
--- a/src/web/styles.css
+++ b/src/web/styles.css
@@ -960,10 +960,11 @@ p {
 }
 
 .client-delete-strike {
+  --client-delete-strike-y: 50%;
   position: absolute;
   inset: 0;
-  z-index: 20;
-  background: radial-gradient(ellipse at center, oklch(0.68 0.15 28 / 0.3) 0%, oklch(0.68 0.15 28 / 0.12) 36%, transparent 74%);
+  z-index: 100;
+  background: radial-gradient(ellipse at 50% var(--client-delete-strike-y), oklch(0.68 0.15 28 / 0.3) 0%, oklch(0.68 0.15 28 / 0.12) 36%, transparent 74%);
   opacity: 0;
   pointer-events: none;
   transform: scaleX(0);
@@ -975,7 +976,7 @@ p {
   position: absolute;
   left: 0;
   right: 0;
-  top: calc(50% - 2px);
+  top: calc(var(--client-delete-strike-y) - 2px);
   height: 4px;
   background: oklch(0.68 0.15 28);
   clip-path: polygon(0 50%, 18% 40%, 50% 28%, 82% 40%, 100% 50%, 82% 60%, 50% 72%, 18% 60%);
@@ -986,6 +987,11 @@ p {
   animation: client-delete-strike 760ms cubic-bezier(0.16, 1, 0.3, 1) both;
 }
 
+.client-deletable-row.is-removing > :not(.client-delete-strike) {
+  position: relative;
+  z-index: 0;
+}
+
 .client-local-rule-enabled {
   width: 24px;
   height: 32px;
diff --git a/test/web/rule-editor-contract.test.js b/test/web/rule-editor-contract.test.js
index 06b4aef..2846b4c 100644
--- a/test/web/rule-editor-contract.test.js
+++ b/test/web/rule-editor-contract.test.js
@@ -23,7 +23,8 @@ test('rule editor add latency stays constant and dirty exits are guarded', () =>
   assert.match(component, /className="client-delete-strike"[\s\S]*onAnimationEnd/);
   assert.match(styles, /\.client-deletable-row\.is-removing > \.client-delete-strike[\s\S]*client-delete-strike/);
   assert.match(styles, /\.client-delete-strike::after[\s\S]*clip-path: polygon\(0 50%[\s\S]*100% 50%/);
-  assert.match(styles, /\.client-delete-strike \{[\s\S]*z-index: 20[\s\S]*radial-gradient/);
+  assert.match(styles, /\.client-delete-strike \{[\s\S]*z-index: 100[\s\S]*radial-gradient/);
+  assert.match(styles, /\.client-deletable-row\.is-removing > :not\(\.client-delete-strike\)[\s\S]*z-index: 0/);
   assert.match(styles, /@keyframes client-delete-strike[\s\S]*scaleX\(0\)[\s\S]*scaleX\(1\)/);
   assert.match(styles, /\.client-local-rule:has\(\.client-rule-type\.is-open\)[\s\S]*z-index: 5/);
 });