Don't break the default sorting passed to the DataGridHeaderCell

marmelab · Aug 10, 2020 · d635a3f · d635a3f
1 parent d747b15
commit d635a3f
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/packages/ra-ui-materialui/src/list/Datagrid.tsx b/packages/ra-ui-materialui/src/list/Datagrid.tsx
@@ -101,13 +101,15 @@ const Datagrid: FC<DatagridProps> = props => {
             event.stopPropagation();
             const newField = event.currentTarget.dataset.field;
             const newOrder =
-                currentSort.field === newField &&
-                event.currentTarget.dataset.order === 'ASC'
-                    ? 'DESC'
-                    : 'ASC';
+                currentSort.field === newField
+                    ? currentSort.order === 'ASC'
+                        ? 'DESC'
+                        : 'ASC'
+                    : event.currentTarget.dataset.order;
+
             setSort(newField, newOrder);
         },
-        [currentSort.field, setSort]
+        [currentSort.field, currentSort.order, setSort]
     );
 
     const handleSelectAll = useCallback(