🔪 unused trace._separators

Author: alexcjohnson

src/traces/carpet/axis_defaults.js

@@ -179,8 +179,6 @@ module.exports = function handleAxisDefaults(containerIn, containerOut, options)
         }
     }
 
-    containerOut._separators = options.fullLayout.separators;
-
     // fill in categories
     containerOut._initialCategories = axType === 'category' ?
         orderedCategories(letter, containerOut.categoryorder, containerOut.categoryarray, options.data) :

src/traces/contour/plot.js

@@ -421,7 +421,7 @@ exports.createLineClip = function(lineContainer, clipLinesForLabels, clips, uid)
 
 exports.labelFormatter = function(contours, colorbar, fullLayout) {
     if(contours.labelformat) {
-        return d3.format(contours.labelformat);
+        return fullLayout._d3locale.numberFormat(contours.labelformat);
     }
     else {
         var formatAxis;
@@ -431,7 +431,6 @@ exports.labelFormatter = function(contours, colorbar, fullLayout) {
         else {
             formatAxis = {
                 type: 'linear',
-                _separators: '.,',
                 _id: 'ycontour',
                 nticks: (contours.end - contours.start) / contours.size,
                 showexponent: 'all',

src/traces/contour/style_defaults.js

@@ -46,9 +46,5 @@ module.exports = function handleStyleDefaults(traceIn, traceOut, coerce, layout,
         coerce('contours.labelformat');
     }
 
-    if(opts.hasHover !== false) {
-        coerce('zhoverformat');
-        // Needed for formatting of hoverlabel if format is not explicitly specified
-        traceOut._separators = layout.separators;
-    }
+    if(opts.hasHover !== false) coerce('zhoverformat');
 };

src/traces/heatmap/style_defaults.js

@@ -9,7 +9,7 @@
 
 'use strict';
 
-module.exports = function handleStyleDefaults(traceIn, traceOut, coerce, layout) {
+module.exports = function handleStyleDefaults(traceIn, traceOut, coerce) {
     var zsmooth = coerce('zsmooth');
     if(zsmooth === false) {
         // ensure that xgap and ygap are coerced only when zsmooth allows them to have an effect.
@@ -18,6 +18,4 @@ module.exports = function handleStyleDefaults(traceIn, traceOut, coerce, layout)
     }
 
     coerce('zhoverformat');
-    // Needed for formatting of hoverlabel if format is not explicitly specified
-    traceOut._separators = layout.separators;
 };