From b4683ef0a1d56f2458fa790229e2a82488344212 Mon Sep 17 00:00:00 2001
From: Ryan-the-zilla <ryan@ryanx.ai>
Date: Wed, 18 Mar 2026 20:52:44 +0000
Subject: [PATCH] fix(testing): improve Test Explorer accessibility and UX

- Disable 'Run All Tests' button when there are zero discovered tests
  (previously only disabled while tests were running)
- Fix hidden row action buttons remaining tabbable by using visibility:hidden
  instead of just opacity:0, removing them from accessibility tree
- Show action buttons when row receives keyboard focus via :focus-within

Fixes #33173, #33198

Wallet: AqE264DnKyJci9kV4t3eYhDtFB3H88HQusWtH5odSqHM
---
 src/components/testing/TestExplorer.tsx | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/src/components/testing/TestExplorer.tsx b/src/components/testing/TestExplorer.tsx
index 2049fe5c..f8e67e9a 100644
--- a/src/components/testing/TestExplorer.tsx
+++ b/src/components/testing/TestExplorer.tsx
@@ -965,7 +965,7 @@ export function TestExplorer(props: TestExplorerProps) {
           <IconButton
             class="test-explorer-action"
             onClick={handleRunAll}
-            disabled={hasRunningTests()}
+            disabled={hasRunningTests() || counts().total === 0}
             tooltip="Run All Tests"
             size="sm"
           >
@@ -1738,12 +1738,15 @@ export function TestExplorer(props: TestExplorerProps) {
           align-items: center;
           gap: 2px;
           opacity: 0;
-          transition: opacity 0.1s;
+          visibility: hidden;
+          transition: opacity 0.1s, visibility 0.1s;
         }
 
         .test-item:hover .test-item-actions,
-        .test-item--selected .test-item-actions {
+        .test-item--selected .test-item-actions,
+        .test-item:focus-within .test-item-actions {
           opacity: 1;
+          visibility: visible;
         }
 
         .test-item-action {