fix bug in goals chart where the sum of all bars could be larger than…

… the expected y-axis window, making the drag handles invisible
jcarver989 · Feb 8, 2017 · 9e14233 · 9e14233
1 parent 5946694
commit 9e14233
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/dist/suave-charts.min.js b/dist/suave-charts.min.js
diff --git a/src/coffeescript/goals_chart.coffee b/src/coffeescript/goals_chart.coffee
@@ -195,7 +195,9 @@ class GoalsChart extends AbstractChart
     tooltipLabels = if data.tooltipLabels? then data.tooltipLabels else ("" for x in data.labels)
     data.bars.forEach((v, i) -> barData.push({ value: v, label: data.labels[i], tooltipLabel: tooltipLabels[i] }))
 
-    @y.domain(data.domain)
+    [min, expectedMax] = data.domain
+    max = Math.max(startingTotal, expectedMax)
+    @y.domain([min, max])
     @x.domain(barData.map((d) -> d.label))
 
     @options.margin.left = @calc.calcLeftMargin(@yAxis, @options.margin.left)