Skip to content

Commit 0e67b1c

Browse files
Pie Charts no longer sort (canada-ca#2408)
* Pie Charts no longer sort This means that the passing component is always displayed first Previous behaviour meant that charts could be ordered differently on the same page.
1 parent 47721ce commit 0e67b1c

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

frontend/src/__tests__/useArcs.test.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,14 +22,14 @@ describe('useArcs', () => {
2222
{
2323
count: 1,
2424
d:
25-
'M-88.02332027547081,47.45413667618336A100,100,0,0,1,-2.915172261501782,-99.95749982210324L-2.91517226150181,-59.92913957905428A60,60,0,0,0,-53.357743433155356,27.43995655465887Z',
25+
'M2.915172261501814,-99.95749982210324A100,100,0,0,1,88.02332027547082,47.45413667618332L53.35774343315538,27.43995655465884A60,60,0,0,0,2.9151722615018025,-59.92913957905428Z',
2626
name: 'bad',
2727
percentage: 1,
2828
},
2929
{
3030
count: 2,
3131
d:
32-
'M2.915172261501814,-99.95749982210324A100,100,0,1,1,-85.10814801396899,52.503363145919955L-50.442571171653555,32.48918302439545A60,60,0,1,0,2.9151722615018025,-59.92913957905428Z',
32+
'M85.10814801396903,52.5033631459199A100,100,0,1,1,-2.915172261501782,-99.95749982210324L-2.91517226150181,-59.92913957905428A60,60,0,1,0,50.442571171653576,32.48918302439542Z',
3333
name: 'good',
3434
percentage: 1,
3535
},
@@ -52,12 +52,12 @@ describe('useArcs', () => {
5252
{
5353
value: 1,
5454
d:
55-
'M-86.60254037844385,50.000000000000036A100,100,0,0,1,-1.8369701987210297e-14,-100L-1.1021821192326178e-14,-60A60,60,0,0,0,-51.961524227066306,30.00000000000002Z',
55+
'M6.123233995736766e-15,-100A100,100,0,0,1,86.60254037844388,49.999999999999986L51.96152422706632,29.99999999999999A60,60,0,0,0,3.67394039744206e-15,-60Z',
5656
},
5757
{
5858
value: 2,
5959
d:
60-
'M6.123233995736766e-15,-100A100,100,0,1,1,-86.60254037844385,50.000000000000036L-51.961524227066306,30.00000000000002A60,60,0,1,0,3.67394039744206e-15,-60Z',
60+
'M86.60254037844388,49.999999999999986A100,100,0,1,1,-1.8369701987210297e-14,-100L-1.1021821192326178e-14,-60A60,60,0,1,0,51.96152422706632,29.99999999999999Z',
6161
},
6262
])
6363
})

frontend/src/useArcs.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ export function useArcs({
77
padAngle = 0,
88
valueAccessor = (d) => d,
99
}) {
10-
const pie = Pie().value(valueAccessor)(data)
10+
const pie = Pie().value(valueAccessor).sort(null)(data)
1111
const arc = Arc()
1212
.innerRadius(innerRadius)
1313
.outerRadius(outerRadius)

0 commit comments

Comments
 (0)