Merge pull request #26 from calebthebrewer/label-z-index

Label z index
helpdotcom · Aug 21, 2015 · 82452b4 · 82452b4
2 parents 9026e13 + 80c07ab
commit 82452b4
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/ampersand-chart.js b/ampersand-chart.js
@@ -1164,6 +1164,16 @@
           .attr('width', lineWidth * 2);
       }.bind(this));
 
+      // move all the text nodes to the bottom of their groups (highest z-index)
+      chart.selectAll('g.ampersand-graph-line-container')
+        .each(function() {
+          // grab all of the text nodes using a remove, also strip the d3 off
+          var texts = d3.select(this).selectAll('text').remove()[0];
+          for (var i = 0, l = texts.length; i < l; i++) {
+            this.appendChild(texts[i]);
+          }
+        });
+
       chart.select('line.ampersand-graph-ground')
         .transition()
         .attr('x2', (2 + data.length) * lineWidth + (data.length - 1) * lineGroupMargin);

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "ampersand-chart",
-  "version": "0.1.21",
+  "version": "0.1.22",
   "description": "Ampersand module for drawing a charts.",
   "main": "ampersand-chart.js",
   "scripts": {