Skip to content

Commit fa57d66

Browse files
kurkleetimberg
authored andcommitted
Backward compatible default fill for radar charts (chartjs#6655)
* Backward compatible fill behavior for radar
1 parent 6ffdfa8 commit fa57d66

File tree

4 files changed

+4
-0
lines changed

4 files changed

+4
-0
lines changed

src/controllers/controller.radar.js

+1
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ defaults._set('radar', {
1414
},
1515
elements: {
1616
line: {
17+
fill: 'start',
1718
tension: 0 // no bezier in radar
1819
}
1920
}

test/fixtures/controller.radar/backgroundColor/scriptable.js

+1
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ module.exports = {
2525
title: false,
2626
elements: {
2727
line: {
28+
fill: true,
2829
backgroundColor: function(ctx) {
2930
var index = (ctx.dataIndex === undefined ? ctx.datasetIndex : ctx.dataIndex);
3031
return index === 0 ? '#ff0000'

test/fixtures/controller.radar/backgroundColor/value.js

+1
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ module.exports = {
2020
title: false,
2121
elements: {
2222
line: {
23+
fill: true,
2324
backgroundColor: '#00ff00'
2425
},
2526
point: {

test/fixtures/controller.radar/borderDash/scriptable.js

+1
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ module.exports = {
2222
title: false,
2323
elements: {
2424
line: {
25+
fill: true,
2526
borderColor: '#00ff00',
2627
borderDash: function(ctx) {
2728
return ctx.datasetIndex === 0 ? [5] : [10];

0 commit comments

Comments
 (0)