archive-gh-me/Chart.js
1
0
Fork 0
mirror of https://github.com/chartjs/Chart.js.git synced 2025-12-08 20:36:08 +00:00
Code Issues Projects Releases Wiki Activity

Fix radar chart horizontal position (#4032)

Browse Source 
Radar chart position is not center horizontally with v2.5.0.

Right and left of `furthestLimits` would be switched wrongly on
this refactoring commit.
e1606f88ed
This commit is contained in:
Hiroshi Shirosaki 2017-03-21 09:39:18 +09:00 committed by Evert Timberg
parent 36ccf40946
commit cffa9447a1
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/scales/scale.radialLinear.js
View File

@ -144,8 +144,8 @@ module.exports = function(Chart) {
// Use this to calculate the offset + change. - Make sure L/R protrusion is at least 0 to stop issues with centre points
var largestPossibleRadius = Math.min(scale.height / 2, scale.width / 2);
var furthestLimits = {
l: scale.width,
r: 0,
r: scale.width,
l: 0,
t: scale.height,
b: 0
};

4
test/specs/scale.radialLinear.tests.js
View File

@ -344,7 +344,7 @@ describe('Test the radial linear scale', function() {
});
expect(chart.scale.drawingArea).toBe(233);
expect(chart.scale.xCenter).toBe(247);
expect(chart.scale.xCenter).toBe(256);
expect(chart.scale.yCenter).toBe(280);
});
@ -393,7 +393,7 @@ describe('Test the radial linear scale', function() {
expect(chart.scale.getDistanceFromCenterForValue(chart.scale.min)).toBe(0);
expect(chart.scale.getDistanceFromCenterForValue(chart.scale.max)).toBe(233);
expect(chart.scale.getPointPositionForValue(1, 5)).toEqual({
x: 261,
x: 270,
y: 275,
});