[IndexTable] Allow row click when onClick is passed in on non-selectable tables

.changeset/odd-swans-teach.md

@@ -0,0 +1,2 @@
+---
+---

polaris-react/src/components/IndexTable/components/Row/Row.tsx

@@ -120,13 +120,14 @@ export const Row = memo(function Row({
     disabled && styles['TableRow-disabled'],
     tone && styles[variationName('tone', tone)],
     !selectable &&
+      !onClick &&
       !primaryLinkElement.current &&
       styles['TableRow-unclickable'],
   );
 
   let handleRowClick;
 
-  if ((!disabled && selectable) || primaryLinkElement.current) {
+  if ((!disabled && selectable) || onClick || primaryLinkElement.current) {
     handleRowClick = (event: React.MouseEvent) => {
       if (rowType === 'subheader') return;
 
@@ -141,7 +142,7 @@ export const Row = memo(function Row({
         return;
       }
 
-      if (primaryLinkElement.current && !selectMode) {
+      if (primaryLinkElement.current && !selectMode && selectable) {
         isNavigating.current = true;
         const {ctrlKey, metaKey} = event.nativeEvent;
 

polaris-react/src/components/IndexTable/components/Row/tests/Row.test.tsx

@@ -444,6 +444,27 @@ describe('<Row />', () => {
     );
   });
 
+  it('fires onClick handler when row has an onclick and is clicked despite no primary link child present and the table not being selectable', () => {
+    const mockOnClick = jest.fn();
+    const row = mountWithTable(
+      <Row {...defaultProps} onClick={mockOnClick}>
+        <th>
+          <a href="/">Child without data-primary-link</a>
+        </th>
+      </Row>,
+      {
+        indexTableProps: {
+          itemCount: 1,
+          selectable: false,
+        },
+      },
+    );
+
+    triggerOnClick(row, 1, defaultEvent);
+
+    expect(mockOnClick).toHaveBeenCalled();
+  });
+
   it('has an undefined tone by default', () => {
     const row = mountWithTable(
       <Row {...defaultProps}>