Skip to content

Commit

Permalink
Revert "Use default value when backgroundColor is undefined (chartjs#…
Browse files Browse the repository at this point in the history
…7950)"

This reverts commit df942bb.
  • Loading branch information
typpo committed Jun 2, 2022
1 parent 778b96e commit d4bc462
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 7 deletions.
3 changes: 1 addition & 2 deletions src/controllers/controller.doughnut.js
Expand Up @@ -25,7 +25,6 @@ defaults._set('doughnut', {
var list = document.createElement('ul');
var data = chart.data;
var datasets = data.datasets;
var globalDefaults = defaults.global;
var labels = data.labels;
var i, ilen, listItem, listItemSpan;

Expand All @@ -34,7 +33,7 @@ defaults._set('doughnut', {
for (i = 0, ilen = datasets[0].data.length; i < ilen; ++i) {
listItem = list.appendChild(document.createElement('li'));
listItemSpan = listItem.appendChild(document.createElement('span'));
listItemSpan.style.backgroundColor = valueOrDefault(datasets[0].backgroundColor[i], globalDefaults.defaultColor);
listItemSpan.style.backgroundColor = datasets[0].backgroundColor[i];
if (labels[i]) {
listItem.appendChild(document.createTextNode(labels[i]));
}
Expand Down
4 changes: 1 addition & 3 deletions src/controllers/controller.polarArea.js
Expand Up @@ -6,7 +6,6 @@ var elements = require('../elements/index');
var helpers = require('../helpers/index');

var resolve = helpers.options.resolve;
var valueOrDefault = helpers.valueOrDefault;

defaults._set('polarArea', {
scale: {
Expand Down Expand Up @@ -36,7 +35,6 @@ defaults._set('polarArea', {
var list = document.createElement('ul');
var data = chart.data;
var datasets = data.datasets;
var globalDefaults = defaults.global;
var labels = data.labels;
var i, ilen, listItem, listItemSpan;

Expand All @@ -45,7 +43,7 @@ defaults._set('polarArea', {
for (i = 0, ilen = datasets[0].data.length; i < ilen; ++i) {
listItem = list.appendChild(document.createElement('li'));
listItemSpan = listItem.appendChild(document.createElement('span'));
listItemSpan.style.backgroundColor = valueOrDefault(datasets[0].backgroundColor[i], globalDefaults.defaultColor);
listItemSpan.style.backgroundColor = datasets[0].backgroundColor[i];
if (labels[i]) {
listItem.appendChild(document.createTextNode(labels[i]));
}
Expand Down
3 changes: 1 addition & 2 deletions src/plugins/plugin.legend.js
Expand Up @@ -80,15 +80,14 @@ defaults._set('global', {
legendCallback: function(chart) {
var list = document.createElement('ul');
var datasets = chart.data.datasets;
var globalDefaults = defaults.global;
var i, ilen, listItem, listItemSpan;

list.setAttribute('class', chart.id + '-legend');

for (i = 0, ilen = datasets.length; i < ilen; i++) {
listItem = list.appendChild(document.createElement('li'));
listItemSpan = listItem.appendChild(document.createElement('span'));
listItemSpan.style.backgroundColor = valueOrDefault(datasets[i].backgroundColor, globalDefaults.defaultColor);
listItemSpan.style.backgroundColor = datasets[i].backgroundColor;
if (datasets[i].label) {
listItem.appendChild(document.createTextNode(datasets[i].label));
}
Expand Down

0 comments on commit d4bc462

Please sign in to comment.