reuse pie.sort

reuse pie.sort in attributes, default true

Author: thierryVergult

src/traces/sunburst/attributes.js

@@ -203,6 +203,7 @@ module.exports = {
             'this option won\'t have any effect and `insidetextfont` would be used.'
         ].join(' ')
     }),
+    sort: pieAttrs.sort,
 
     domain: domainAttrs({name: 'sunburst', trace: true, editType: 'calc'})
 };

src/traces/sunburst/calc.js

@@ -192,7 +192,7 @@ exports.calc = function(gd, trace) {
     if(failed) return;
 
     // TODO add way to sort by height also?
-    if ( fullLayout[ 'sort']) {
+    if ( trace.sort) {
       hierarchy.sort(function(a, b) { return b.value - a.value; });
     }
 

src/traces/sunburst/defaults.js

@@ -73,6 +73,8 @@ module.exports = function supplyDefaults(traceIn, traceOut, defaultColor, layout
 
     coerce('insidetextorientation');
 
+    coerce('sort');
+
     handleDomainDefaults(traceOut, layout, coerce);
 
     // do not support transforms for now

src/traces/sunburst/layout_attributes.js

@@ -35,15 +35,5 @@ module.exports = {
             'Colors provided in the trace, using `marker.colors`, are never',
             'extended.'
         ].join(' ')
-    },
-    sort: {
-      valType: 'boolean',
-      dflt: true,
-      role: 'style',
-      editType: 'calc',
-      description: [
-          'sort on values.',
-          'default true'
-      ].join(' ')
     }
 };

src/traces/sunburst/layout_defaults.js

@@ -17,5 +17,4 @@ module.exports = function supplyLayoutDefaults(layoutIn, layoutOut) {
     }
     coerce('sunburstcolorway', layoutOut.colorway);
     coerce('extendsunburstcolors');
-    coerce('sort');
 };