fixed case insensitive sorting for Ktable

learningequality · Dec 7, 2024 · a1f07b0 · a1f07b0
1 parent 20f9350
commit a1f07b0
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/lib/KTable/useSorting/index.js b/lib/KTable/useSorting/index.js
@@ -29,7 +29,7 @@ export default function useSorting(headers, rows, defaultSort, disableBuiltinSor
   const sortedRows = computed(() => {
     if (disableBuiltinSorting.value) return rows.value;
 
-    // No sort key or value has been explicity set till now
+    // No sort key or value has been explicitly set till now
     if (sortKey.value === null || sortOrder.value === null) {
       if (defaultSort.value.index === -1) return rows.value;
 
@@ -40,7 +40,13 @@ export default function useSorting(headers, rows, defaultSort, disableBuiltinSor
       );
     }
 
-    return _.orderBy(rows.value, [row => row[sortKey.value]], [sortOrder.value]);
+    return _.orderBy(rows.value, [row => {
+      const value = row[sortKey.value];
+      if (typeof value === 'string') {
+        return value.toString().toLocaleLowerCase();
+      }
+      return value;
+    }], [sortOrder.value]);
   });
 
   /**
@@ -85,4 +91,4 @@ export default function useSorting(headers, rows, defaultSort, disableBuiltinSor
     handleSort,
     getAriaSort,
   };
-}
+}