From 89effa74e2fcb3e631a08102bed51744f705975c Mon Sep 17 00:00:00 2001
From: Matthew Somerville <matthew-github@dracos.co.uk>
Date: Tue, 19 Dec 2017 17:13:25 +0000
Subject: Fix colour wrapping bug, where it only added red.

As colours.length increased as colours were added,
it would always return the first entry.
---
 web/js/dashboard.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'web/js/dashboard.js')

diff --git a/web/js/dashboard.js b/web/js/dashboard.js
index a6e06e048..b2a87c142 100644
--- a/web/js/dashboard.js
+++ b/web/js/dashboard.js
@@ -179,8 +179,8 @@ $(function(){
             rowValues.push( parseInt($(this).find('td').text(), 10) );
         });
 
-        for (var i=colours.length; i<rowLabels.length; i++) {
-            colours[i] = colours[i % colours.length];
+        for (var l=colours.length, i=l; i<rowLabels.length; i++) {
+            colours[i] = colours[i % l];
         }
 
         var barChart = new Chart($canvas, {
-- 
cgit v1.2.3