Skip to content

Commit 4e79058

Browse files
author
cristian-ungureanu
authored
Merge pull request #858 from Codeinwp/bugfix/318
Fix chartjs tooltip issue after upgrading library
2 parents 9264bb5 + 7d8931b commit 4e79058

File tree

2 files changed

+14
-14
lines changed

2 files changed

+14
-14
lines changed

js/lib/chartjs.min.js

Lines changed: 9 additions & 12 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

js/render-chartjs.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -173,12 +173,12 @@
173173
}
174174

175175
if(typeof settings['fontName'] !== 'undefined' && settings['fontName'] !== ''){
176-
Chart.defaults.global.defaultFontFamily = settings['fontName'];
176+
Chart.defaults.font.family = settings['fontName'];
177177
delete settings['fontName'];
178178
}
179179

180180
if(typeof settings['fontSize'] !== 'undefined' && settings['fontSize'] !== ''){
181-
Chart.defaults.global.defaultFontSize = parseInt(settings['fontSize']);
181+
Chart.defaults.font.size = parseInt(settings['fontSize']);
182182
delete settings['fontSize'];
183183
}
184184

@@ -337,6 +337,9 @@
337337
}
338338
for(var i in settings.series[j]){
339339
var $attribute = {};
340+
if ( settings.series[j].backgroundColor == '' ) {
341+
delete settings.series[j].backgroundColor;
342+
}
340343
$attribute[i] = settings.series[j][i];
341344
$.extend($attributes, $attribute);
342345
}

0 commit comments

Comments
 (0)