Merge pull request #130 from Anjalbaral/multi-order-table-sort

Some enhancements to mult-order table sort
Packet-Clearing-House · Jun 14, 2024 · b9fa8c0 · b9fa8c0
2 parents c21fb52 + f40ace2
commit b9fa8c0
Show file tree

Hide file tree

Showing 7 changed files with 142 additions and 37 deletions.
diff --git a/dist/maptable.css b/dist/maptable.css
diff --git a/dist/maptable.js b/dist/maptable.js
diff --git a/dist/maptable.js.map b/dist/maptable.js.map
diff --git a/dist/maptable.min.js b/dist/maptable.min.js
diff --git a/docs/maptable.js b/docs/maptable.js
diff --git a/docs/maptable.min.js b/docs/maptable.min.js
diff --git a/src/components/Table.js b/src/components/Table.js
@@ -324,7 +324,8 @@ export default class Table {
     const primarySort = this.sorting[0].key || '';
     if (data[index] && data[index + 1]) {
       if (data[index][primarySort] && data[index + 1][primarySort]) {
-        return data[index][primarySort].toLowerCase() !== data[index + 1][primarySort].toLowerCase();
+        // stringify everything before .toLowerCase()
+        return `${data[index][primarySort]}`.toLowerCase() !== `${data[index + 1][primarySort]}`.toLowerCase();
       }
     }
     return false;