Merge pull request #61 from TEA-ebook/dev/allow-many-theme-colors

ratibus · ratibus · commit b0df9fae0be9 · 2015-01-22T11:37:51.000+01:00
Allow many theme colors
diff --git a/jquery.highchartTable.js b/jquery.highchartTable.js
@@ -315,7 +315,8 @@
       var lineShadow  = $table.data('graph-line-shadow');
       var lineWidth   = $table.data('graph-line-width') || 2;
 
-      for(var i=0; i<9; i++) {
+      var nbOfColors = Math.max(defaultColors.length, themeColors.length);
+      for(var i=0; i < nbOfColors; i++) {
         var dataname = 'graph-color-' + (i+1);
         colors.push(typeof $table.data(dataname) != 'undefined' ? $table.data(dataname) : typeof themeColors[i] != 'undefined' ? themeColors[i] : defaultColors[i]);
       }

Original file line number	Diff line number	Diff line change
`@@ -315,7 +315,8 @@`
`315`	`315`	`var lineShadow = $table.data('graph-line-shadow');`
`316`	`316`	`var lineWidth = $table.data('graph-line-width') \|\| 2;`
`317`	`317`
`318`		`- for(var i=0; i<9; i++) {`
	`318`	`+ var nbOfColors = Math.max(defaultColors.length, themeColors.length);`
	`319`	`+ for(var i=0; i < nbOfColors; i++) {`
`319`	`320`	`var dataname = 'graph-color-' + (i+1);`
`320`	`321`	`colors.push(typeof $table.data(dataname) != 'undefined' ? $table.data(dataname) : typeof themeColors[i] != 'undefined' ? themeColors[i] : defaultColors[i]);`
`321`	`322`	`}`