Skip to content

Commit b3c1e7a

Browse files
authored
Merge pull request #4268 from jtvd78/check-for-typed-array-in-minextend
Check for typed array in minExtend
2 parents 25fe1c9 + 7f3b2bb commit b3c1e7a

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

src/lib/index.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -666,6 +666,8 @@ lib.minExtend = function(obj1, obj2) {
666666
} else {
667667
objOut[k] = v.slice(0, arrayLen);
668668
}
669+
} else if(lib.isTypedArray(v)) {
670+
objOut[k] = v.subarray(0, arrayLen);
669671
} else if(v && (typeof v === 'object')) objOut[k] = lib.minExtend(obj1[k], obj2[k]);
670672
else objOut[k] = v;
671673
}

test/jasmine/tests/scatter_test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1121,7 +1121,7 @@ describe('end-to-end scatter tests', function() {
11211121

11221122
var legendPts = d3.select('.legend').selectAll('.scatterpts');
11231123
expect(legendPts.size()).toBe(1, '# legend items');
1124-
expect(getColor(legendPts.node())).toBe('rgb(0, 0, 0)', 'legend pt color');
1124+
expect(getColor(legendPts.node())).toBe('rgb(0, 255, 0)', 'legend pt color');
11251125
expect(getMarkerSize(legendPts.node())).toBe(16, 'legend pt size');
11261126
})
11271127
.catch(failTest)

0 commit comments

Comments
 (0)