Merge pull request #1626 from plotly/add-class-to-text-trace

Add textpoint class to scatter text points

Author: rreusser

src/traces/scatter/plot.js

@@ -460,6 +460,7 @@ function plotOne(gd, idx, plotinfo, cdscatter, cdscatterAll, element, transition
         });
 
         join.selectAll('text')
+            .classed('textpoint', true)
             .call(Drawing.textPointStyle, trace)
             .each(function(d) {
 

test/jasmine/tests/scatter_test.js

@@ -372,6 +372,17 @@ describe('end-to-end scatter tests', function() {
 
         }).catch(fail).then(done);
     });
+
+    it('adds "textpoint" class to scatter text points', function(done) {
+        Plotly.plot(gd, [{
+            mode: 'text',
+            x: [1, 2, 3],
+            y: [2, 3, 4],
+            text: ['a', 'b', 'c']
+        }]).then(function() {
+            expect(Plotly.d3.selectAll('.textpoint').size()).toBe(3);
+        }).catch(fail).then(done);
+    });
 });
 
 describe('scatter hoverPoints', function() {