From fbd5c5a93dd8ae321e654c982542f6e80d2bf466 Mon Sep 17 00:00:00 2001
From: Johan Persson <johanopersson@gmail.com>
Date: Tue, 10 Mar 2026 10:01:45 +0100
Subject: [PATCH] fix(ui): only show pointer cursor on interactive Table rows

Row always passed onAction to React Aria even when no handler or
href was set, causing all rows to appear interactive. Now onAction
is only passed when there is an actual interaction. CSS explicitly
sets cursor: default on rows and scopes cursor: pointer to rows
with href, selection mode, or pressable state.

Signed-off-by: Johan Persson <johanopersson@gmail.com>
---
 .../fix-table-row-cursor-non-interactive.md   |  7 ++++++
 .../ui/src/components/Table/Table.module.css  |  3 +++
 .../src/components/Table/components/Row.tsx   | 23 ++++++++++---------
 3 files changed, 22 insertions(+), 11 deletions(-)
 create mode 100644 .changeset/fix-table-row-cursor-non-interactive.md

diff --git a/.changeset/fix-table-row-cursor-non-interactive.md b/.changeset/fix-table-row-cursor-non-interactive.md
new file mode 100644
index 0000000000..cf83ea6631
--- /dev/null
+++ b/.changeset/fix-table-row-cursor-non-interactive.md
@@ -0,0 +1,7 @@
+---
+'@backstage/ui': patch
+---
+
+Fixed Table rows showing a pointer cursor when not interactive. Rows now only show `cursor: pointer` when they have an `href`, are selectable, or are pressable.
+
+**Affected components:** Table
diff --git a/packages/ui/src/components/Table/Table.module.css b/packages/ui/src/components/Table/Table.module.css
index 0faa256e06..9663fd9aa8 100644
--- a/packages/ui/src/components/Table/Table.module.css
+++ b/packages/ui/src/components/Table/Table.module.css
@@ -83,6 +83,7 @@
   .bui-TableRow {
     border-bottom: 1px solid var(--bui-border-2);
     transition: color 0.2s ease-in-out;
+    cursor: default;
 
     &:hover {
       background-color: var(--bui-bg-tint-hover);
@@ -100,6 +101,8 @@
       background-color: var(--bui-bg-tint-disabled);
     }
 
+    &[data-href],
+    &[data-selection-mode],
     &[data-react-aria-pressable='true'] {
       cursor: pointer;
     }
diff --git a/packages/ui/src/components/Table/components/Row.tsx b/packages/ui/src/components/Table/components/Row.tsx
index 54e2ea3a77..3da46b3eaa 100644
--- a/packages/ui/src/components/Table/components/Row.tsx
+++ b/packages/ui/src/components/Table/components/Row.tsx
@@ -37,14 +37,18 @@ export function Row<T extends object>(props: RowProps<T>) {
   );
   const { classes, columns, children, href } = ownProps;
   const hasInternalHref = !!href && !isExternalLink(href);
+  const hasInteraction = !!restProps.onAction || !!href;
 
-  const handlePress = () => {
-    if (href) {
-      analytics.captureEvent('click', href, {
-        attributes: { to: String(href) },
-      });
-    }
-  };
+  const handlePress = hasInteraction
+    ? () => {
+        restProps.onAction?.();
+        if (href) {
+          analytics.captureEvent('click', href, {
+            attributes: { to: String(href) },
+          });
+        }
+      }
+    : undefined;
 
   let { selectionBehavior, selectionMode } = useTableOptions();
 
@@ -70,10 +74,7 @@ export function Row<T extends object>(props: RowProps<T>) {
         className={classes.root}
         data-react-aria-pressable={hasInternalHref ? 'true' : undefined}
         {...restProps}
-        onAction={() => {
-          restProps.onAction?.();
-          handlePress();
-        }}
+        onAction={handlePress}
       >
         {content}
       </ReactAriaRow>