Merge pull request #3494 from Kunnu01/fix/sortable

[RFR] add check for sortable in shouldUpdate
marmelab · Aug 7, 2019 · a178e98 · a178e98
2 parents 4b05bb6 + 87ed11d
commit a178e98
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/packages/ra-ui-materialui/src/list/DatagridHeaderCell.js b/packages/ra-ui-materialui/src/list/DatagridHeaderCell.js
@@ -97,7 +97,8 @@ const enhance = compose(
         (props, nextProps) =>
             props.isSorting !== nextProps.isSorting ||
             (nextProps.isSorting &&
-                props.currentSort.order !== nextProps.currentSort.order)
+                props.currentSort.order !== nextProps.currentSort.order) ||
+            (nextProps.isSorting && props.sortable !== nextProps.sortable)
     ),
     translate,
     withStyles(styles)