rename key to

Author: archmoj

src/traces/sunburst/attributes.js

@@ -158,7 +158,7 @@ module.exports = {
             'value',
             'current path',
             'percent root',
-            'percent visible',
+            'percent entry',
             'percent parent'
         ],
         extras: ['none'],
@@ -182,7 +182,7 @@ module.exports = {
             'name',
             'current path',
             'percent root',
-            'percent visible',
+            'percent entry',
             'percent parent'
         ],
         dflt: 'label+text+value+name'

src/traces/sunburst/constants.js

@@ -12,12 +12,20 @@ module.exports = {
     CLICK_TRANSITION_TIME: 750,
     CLICK_TRANSITION_EASING: 'linear',
     eventDataKeys: [
+        // string
+        'root',
+        'entry',
+        // 'parent', // no need to add parent here which is added somewhere else!
         'currentPath',
+
+        // percentages i.e. ratios
         'percentRoot',
-        'percentVisible',
+        'percentEntry',
         'percentParent',
-        'root',
-        'visible',
-        'parent'
+
+        // sums
+        'sumRoot',
+        'sumEntry',
+        'sumParent'
     ]
 };

src/traces/sunburst/fx.js

@@ -115,10 +115,10 @@ module.exports = function attachFxHandlers(sliceTop, entry, gd, cd, opts) {
 
             var ref1 = entry;
             if(ref1 && getVal(ref1)) {
-                hoverPt.percentVisible = pt.percentVisible = val / getVal(ref1);
-                hoverPt.visible = pt.visible = helpers.getLabelString(ref1.data.data.label);
-                if(hasFlag('percent visible')) {
-                    tx = helpers.formatPercent(hoverPt.percentVisible, separators) + ' of ' + hoverPt.visible;
+                hoverPt.percentEntry = pt.percentEntry = val / getVal(ref1);
+                hoverPt.entry = pt.entry = helpers.getLabelString(ref1.data.data.label);
+                if(hasFlag('percent entry')) {
+                    tx = helpers.formatPercent(hoverPt.percentEntry, separators) + ' of ' + hoverPt.entry;
                     insertPercent();
                 }
             }

src/traces/sunburst/plot.js

@@ -519,7 +519,7 @@ exports.formatSliceLabel = function(pt, entry, trace, cd, fullLayout) {
 
         var nPercent = 0;
         if(hasFlag('percent parent')) nPercent++;
-        if(hasFlag('percent visible')) nPercent++;
+        if(hasFlag('percent entry')) nPercent++;
         if(hasFlag('percent root')) nPercent++;
         var hasMultiplePercents = nPercent > 1;
 
@@ -541,9 +541,9 @@ exports.formatSliceLabel = function(pt, entry, trace, cd, fullLayout) {
                 ref = pt.parent;
                 makePercent('parent');
             }
-            if(hasFlag('percent visible') && pt.parent) {
+            if(hasFlag('percent entry') && pt.parent) {
                 ref = entry;
-                makePercent('visible');
+                makePercent('entry');
             }
             if(hasFlag('percent root') && (pt.parent || helpers.isLeaf(pt))) {
                 ref = hierarchy;
@@ -580,11 +580,11 @@ exports.formatSliceLabel = function(pt, entry, trace, cd, fullLayout) {
     }
 
     ref = entry;
-    obj.percentVisible = calcPercent();
-    obj.percentVisibleLabel = helpers.formatPercent(
-        obj.percentVisible, separators
+    obj.percentEntry = calcPercent();
+    obj.percentEntryLabel = helpers.formatPercent(
+        obj.percentEntry, separators
     );
-    obj.visible = ref.data.data.label;
+    obj.entry = ref.data.data.label;
 
     ref = hierarchy;
     obj.percentRoot = calcPercent();

src/traces/treemap/constants.js

@@ -12,13 +12,21 @@ module.exports = {
     CLICK_TRANSITION_TIME: 750,
     CLICK_TRANSITION_EASING: 'poly',
     eventDataKeys: [
+        // string
+        'root',
+        'entry',
+        // 'parent', // no need to add parent here which is added somewhere else!
         'currentPath',
+
+        // percentages i.e. ratios
         'percentRoot',
-        'percentVisible',
+        'percentEntry',
         'percentParent',
-        'root',
-        'visible',
-        'parent'
+
+        // sums
+        'sumRoot',
+        'sumEntry',
+        'sumParent'
     ],
     gapWithPathbar: 1 // i.e. one pixel
 };