Merge pull request #8823 from marmelab/Datagrid-rowClick-false

Fix Datagrid rowClick PropTypes

Author: djhi

packages/ra-ui-materialui/src/list/datagrid/Datagrid.stories.tsx

@@ -403,3 +403,14 @@ export const ErrorInFetch = () => (
         </ListContextProvider>
     </MemoryRouter>
 );
+
+export const RowClickFalse = () => (
+    <Wrapper>
+        <Datagrid rowClick={false}>
+            <TextField source="id" />
+            <TextField source="title" />
+            <TextField source="author" />
+            <TextField source="year" />
+        </Datagrid>
+    </Wrapper>
+);

packages/ra-ui-materialui/src/list/datagrid/Datagrid.tsx

@@ -309,7 +309,12 @@ Datagrid.propTypes = {
     onSelect: PropTypes.func,
     onToggleItem: PropTypes.func,
     resource: PropTypes.string,
-    rowClick: PropTypes.oneOfType([PropTypes.string, PropTypes.func]),
+    // @ts-ignore
+    rowClick: PropTypes.oneOfType([
+        PropTypes.string,
+        PropTypes.func,
+        PropTypes.bool,
+    ]),
     rowStyle: PropTypes.func,
     selectedIds: PropTypes.arrayOf(PropTypes.any),
     setSort: PropTypes.func,

packages/ra-ui-materialui/src/list/datagrid/DatagridBody.tsx

@@ -73,7 +73,12 @@ DatagridBody.propTypes = {
     onToggleItem: PropTypes.func,
     resource: PropTypes.string,
     row: PropTypes.element,
-    rowClick: PropTypes.oneOfType([PropTypes.string, PropTypes.func]),
+    // @ts-ignore
+    rowClick: PropTypes.oneOfType([
+        PropTypes.string,
+        PropTypes.func,
+        PropTypes.bool,
+    ]),
     rowStyle: PropTypes.func,
     selectedIds: PropTypes.arrayOf(PropTypes.any),
     styles: PropTypes.object,

packages/ra-ui-materialui/src/list/datagrid/DatagridRow.tsx

@@ -241,7 +241,11 @@ DatagridRow.propTypes = {
     record: PropTypes.object,
     resource: PropTypes.string,
     // @ts-ignore
-    rowClick: PropTypes.oneOfType([PropTypes.string, PropTypes.func]),
+    rowClick: PropTypes.oneOfType([
+        PropTypes.string,
+        PropTypes.func,
+        PropTypes.bool,
+    ]),
     selected: PropTypes.bool,
     style: PropTypes.object,
     selectable: PropTypes.bool,