add support for <b>, <i> and <em> in "name" hover labels

Author: etpinard

Diff for: src/components/fx/hover.js

@@ -927,7 +927,7 @@ function createHoverText(hoverData, opts, gd) {
         if(d.name) {
             name = svgTextUtils.plainText(d.name || '', {
                 len: d.nameLength,
-                allowedTags: ['br', 'sub', 'sup']
+                allowedTags: ['br', 'sub', 'sup', 'b', 'i', 'em']
             });
         }
 

Diff for: test/jasmine/tests/svg_text_utils_test.js

@@ -504,5 +504,13 @@ describe('svg+text utils', function() {
             expect(fn(sIn, {allowedTags: ['sup']})).toBe('ThisIsDATA<sup>300</sup>');
             expect(fn(sIn, {allowedTags: ['br', 'sup']})).toBe('ThisIs<br>DATA<sup>300</sup>');
         });
+
+        it('should work in various cases w/ <b>, <i> and <em>', function() {
+            var sIn = '<i>ThisIs</i><b>DATA</b><em>300</em>';
+
+            expect(fn(sIn)).toBe('ThisIsDATA300');
+            expect(fn(sIn, {allowedTags: ['i', 'b', 'em']})).toBe('<i>ThisIs</i><b>DATA</b><em>300</em>');
+            expect(fn(sIn, {len: 10, allowedTags: ['i', 'b', 'em']})).toBe('<i>ThisIs</i>D...');
+        });
     });
 });