Skip to content
This repository was archived by the owner on Mar 23, 2025. It is now read-only.

Commit e568ec8

Browse files
committed
Fix the unexpected sorting behavior on Chrome
1 parent ac3df03 commit e568ec8

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

public/js/sort-items.js

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -50,10 +50,10 @@ $(() => {
5050
sortedKeys.sort((a, b) => {
5151
// sort by frequency of the key first
5252
if (keyRange[a][2] !== keyRange[b][2]) {
53-
return keyRange[a][2] < keyRange[b][2];
53+
return (keyRange[a][2] < keyRange[b][2]) ? 1 : -1;
5454
}
5555
// then sort by range of the key
56-
return (keyRange[a][1] - keyRange[a][0]) < (keyRange[b][1] - keyRange[b][0]);
56+
return ((keyRange[a][1] - keyRange[a][0]) < (keyRange[b][1] - keyRange[b][0])) ? 1 : -1;
5757
});
5858

5959
console.log(sortedKeys);
@@ -70,7 +70,7 @@ $(() => {
7070
return -1;
7171
if (a.numbers[key] === b.numbers[key])
7272
continue;
73-
return a.numbers[key] > b.numbers[key];
73+
return (a.numbers[key] > b.numbers[key]) ? 1 : -1;
7474
}
7575
return 0;
7676
});
@@ -102,12 +102,11 @@ $(() => {
102102
res = ap > bp;
103103
}
104104
if (dir === 'up')
105-
return res;
105+
return res ? 1 : -1;
106106
else
107-
return !res;
107+
return !res ? 1 : -1;
108108
});
109109
}
110-
var html = '';
111110
$('#item-container').append(items);
112111
};
113112

0 commit comments

Comments
 (0)