don't default sort by to first column (#2653)

apache · Apr 20, 2017 · 84fa0d1 · 84fa0d1
1 parent 938e13a
commit 84fa0d1
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/superset/assets/javascripts/components/FilterableTable/FilterableTable.jsx b/superset/assets/javascripts/components/FilterableTable/FilterableTable.jsx
@@ -41,7 +41,7 @@ export default class FilterableTable extends PureComponent {
       .reduce((curr, next) => curr + next);
 
     this.state = {
-      sortBy: props.orderedColumnKeys[0],
+      sortBy: null,
       sortDirection: SortDirection.ASC,
       fitted: false,
     };
@@ -142,9 +142,11 @@ export default class FilterableTable extends PureComponent {
       sortedAndFilteredList = this.list.filter(row => this.hasMatch(filterText, row));
     }
     // sort list
-    sortedAndFilteredList = sortedAndFilteredList
+    if (sortBy) {
+      sortedAndFilteredList = sortedAndFilteredList
       .sortBy(item => item[sortBy])
       .update(list => sortDirection === SortDirection.DESC ? list.reverse() : list);
+    }
 
     const rowGetter = ({ index }) => this.getDatum(sortedAndFilteredList, index);