fix bug where react-style frameworks might call draw() on a node befo…

…re it being actually attached to the DOM - now we wait & poll for the chart to be in the DOM before attempting to draw it

src/coffeescript/abstract_chart.coffee

@@ -8,6 +8,12 @@ class AbstractChart
     @listenerBound = false
     @axisLabelPadding = 10 
 
+  waitToBeInDom: (func) -> 
+    if document.body.contains(@svg.container[0][0])
+      func()
+    else
+      setTimeout(func, 100)
+
   draw: () ->
     unless @listenerBound
       window.addEventListener("resize", @render)

src/coffeescript/bar_chart.coffee

@@ -83,6 +83,9 @@ class BarChart extends AbstractChart
 
   draw: (data) ->
     super()
+    @waitToBeInDom(() => @drawInternal(data))
+
+  drawInternal: (data) ->
     normalizedBars = ((if Object.prototype.toString.call(d) == "[object Array]" then d else [d]) for d in data.bars)
 
     @y.domain([0, d3.max(d3.merge(normalizedBars, (bars) -> d3.max(bars)))])

src/coffeescript/donut_chart.coffee

@@ -24,6 +24,9 @@ class DonutChart extends AbstractChart
 
   draw: (data) =>
     super()
+    @waitToBeInDom(() => @drawInternal(data))
+
+  drawInternal: (data) =>
     s = (sum, d) -> sum + d[1]
     arcSelection = @svg.chart.selectAll(".arc").data(@pie(data))
     @enter = arcSelection.enter()

src/coffeescript/histogram.coffee

@@ -73,6 +73,9 @@ class Histogram extends AbstractChart
 
   draw: (data) ->
     super()
+    @waitToBeInDom(() => @drawInternal(data))
+
+  drawInternal: (data) ->
     hist = d3.layout.histogram()
     if @options.domain
       @x.domain(@options.domain)

src/coffeescript/line_chart.coffee

@@ -116,6 +116,9 @@ class LineChart extends AbstractChart
 
   draw: (data) ->
     super()
+    @waitToBeInDom(() => @drawInternal(data))
+
+  drawInternal: (data) ->
     { labels, lines, xDomain, yDomain } = @formatData(data, @options.stack)
 
     if @options.xScale == "ordinal"

src/coffeescript/scatterplot.coffee

@@ -3,6 +3,10 @@ class ScatterPlot extends AbstractChart
     super(selector, options)
 
   draw: (data) ->
+    super()
+    @waitToBeInDom(() => @drawInternal(data))
+
+  drawInternal: (data) ->
     @x.domain d3.extent(data, extractX)
     @y.domain [0, d3.max(data, extractY)]