revisit pathbar divider default

Author: archmoj

src/traces/treemap/attributes.js

@@ -211,12 +211,11 @@ module.exports = {
                 '/',
                 '\\'
             ],
+            dflt: '>',
             role: 'style',
             editType: 'plot',
             description: [
-                'Determines which divider is used between labels.',
-                'With *top* `pathbar.side` it is defaulted to */*; and',
-                'with *bottom* `pathbar.side` it is defaulted to *\\*.'
+                'Determines which divider is used between labels.'
             ].join(' ')
         },
 

src/traces/treemap/defaults.js

@@ -105,8 +105,8 @@ module.exports = function supplyDefaults(traceIn, traceOut, defaultColor, layout
         // This works even for multi-line labels as treemap pathbar trim out line breaks
         coerce('pathbar.thickness', traceOut.pathbar.textfont.size + 2 * TEXTPAD);
 
-        var side = coerce('pathbar.side');
-        coerce('pathbar.divider', side === 'top' ? '/' : '\\');
+        coerce('pathbar.side');
+        coerce('pathbar.divider');
     }
 
     handleDomainDefaults(traceOut, layout, coerce);

test/image/baselines/treemap_level-depth.png

@@ -253,7 +253,7 @@ describe('Test treemap defaults:', function() {
         expect(fullData[0].pathbar.textfont.size).toBe(12);
         expect(fullData[0].pathbar.thickness).toBe(18);
         expect(fullData[0].pathbar.side).toBe('top');
-        expect(fullData[0].pathbar.divider).toBe('/');
+        expect(fullData[0].pathbar.divider).toBe('>');
     });
 
     it('should default *pathbar* sizes and styles to layout', function() {
@@ -268,16 +268,6 @@ describe('Test treemap defaults:', function() {
         expect(fullData[0].pathbar.textfont.size).toBe(24);
         expect(fullData[0].pathbar.thickness).toBe(30);
     });
-
-    it('should default *pathbar.divider* in respect to *path.side*', function() {
-        _supply([
-            {labels: [1], parents: [''], pathbar: {side: 'top'}},
-            {labels: [1], parents: [''], pathbar: {side: 'bottom'}}
-        ]);
-
-        expect(fullData[0].pathbar.divider).toBe('/', 'when path.barside is top');
-        expect(fullData[1].pathbar.divider).toBe('\\', 'when path.barside is bottom');
-    });
 });
 
 describe('Test treemap calc:', function() {