Fix inconsistent aspect ratio

chartjs · Mar 26, 2017 · ba78d9e · ba78d9e
1 parent f3816b5
commit ba78d9e
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 6 deletions.
diff --git a/src/controllers/controller.doughnut.js b/src/controllers/controller.doughnut.js
@@ -12,7 +12,6 @@ module.exports = function(Chart) {
 			// Boolean - Whether we animate scaling the Doughnut from the centre
 			animateScale: false
 		},
-		aspectRatio: 1,
 		hover: {
 			mode: 'single'
 		},

diff --git a/src/controllers/controller.polarArea.js b/src/controllers/controller.polarArea.js
@@ -29,7 +29,6 @@ module.exports = function(Chart) {
 		},
 
 		startAngle: -0.5 * Math.PI,
-		aspectRatio: 1,
 		legendCallback: function(chart) {
 			var text = [];
 			text.push('<ul class="' + chart.id + '-legend">');

diff --git a/src/controllers/controller.radar.js b/src/controllers/controller.radar.js
@@ -5,7 +5,6 @@ module.exports = function(Chart) {
 	var helpers = Chart.helpers;
 
 	Chart.defaults.radar = {
-		aspectRatio: 1,
 		scale: {
 			type: 'radialLinear'
 		},

diff --git a/test/specs/platform.dom.tests.js b/test/specs/platform.dom.tests.js
@@ -107,13 +107,13 @@ describe('Platform.dom', function() {
 				}
 			}, {
 				canvas: {
-					style: 'width: 425px'
+					style: 'width: 212px'
 				}
 			});
 
 			expect(chart).toBeChartOfSize({
-				dw: 425, dh: 425,
-				rw: 425, rh: 425,
+				dw: 212, dh: 106,
+				rw: 212, rh: 106,
 			});
 		});