status-im/sourcecred
2
0
Fork 0
mirror of https://github.com/status-im/sourcecred.git synced 2025-02-25 18:55:21 +00:00
Code Issues Projects Releases Wiki Activity

Remove unnecessary reversals in sort routine

Browse Source 
We were sorting low-to-high, and then reversing. We can just sort
high-to-low.

Test plan: No behavior change (confirm by interacting with the Cred
Explorer).

Paired with @wchargin
This commit is contained in:
Dandelion Mané 2018-05-10 16:34:48 -07:00
parent 8a4b9592b1
commit 880b0099e9
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/app/credExplorer/pagerankTable.js
View File

@ -123,9 +123,8 @@ export class PagerankTable extends React.Component<Props, State> {
.sort((a, b) => {
const x = pagerankResult.get(a.address).probability;
const y = pagerankResult.get(b.address).probability;
return x - y;
})
.reverse();
return y - x;
});
return (
<table>
<thead>
@ -202,9 +201,8 @@ class RecursiveTable extends React.Component<RTProps, RTState> {
.sort((a, b) => {
const x = pagerankResult.get(a.address).probability;
const y = pagerankResult.get(b.address).probability;
return x - y;
return y - x;
})
.reverse()
.map(({address}) => (
<RecursiveTable
depth={depth + 1}