Radar charts should fill to the center by default (#7446)

* Radar charts should fill to the center by default * Update tests for new defaults
chartjs · Sep 1, 2020 · e26f7b9 · e26f7b9
1 parent 2b57660
commit e26f7b9
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 3 deletions.
diff --git a/src/controllers/controller.radar.js b/src/controllers/controller.radar.js
@@ -13,6 +13,7 @@ defaults.set('radar', {
 	},
 	elements: {
 		line: {
+			fill: 'start',
 			tension: 0 // no bezier in radar
 		}
 	}

diff --git a/test/fixtures/controller.radar/backgroundColor/scriptable.js b/test/fixtures/controller.radar/backgroundColor/scriptable.js
@@ -30,7 +30,8 @@ module.exports = {
 						return index === 0 ? '#ff0000'
 							: index === 1 ? '#00ff00'
 							: '#ff00ff';
-					}
+					},
+					fill: true,
 				},
 				point: {
 					backgroundColor: '#0000ff',

diff --git a/test/fixtures/controller.radar/backgroundColor/value.js b/test/fixtures/controller.radar/backgroundColor/value.js
@@ -20,7 +20,8 @@ module.exports = {
 			title: false,
 			elements: {
 				line: {
-					backgroundColor: '#00ff00'
+					backgroundColor: '#00ff00',
+					fill: true,
 				},
 				point: {
 					radius: 10

diff --git a/test/fixtures/controller.radar/borderDash/scriptable.js b/test/fixtures/controller.radar/borderDash/scriptable.js
@@ -25,7 +25,8 @@ module.exports = {
 					borderColor: '#00ff00',
 					borderDash: function(ctx) {
 						return ctx.datasetIndex === 0 ? [5] : [10];
-					}
+					},
+					fill: true,
 				},
 				point: {
 					radius: 10