add category 'noSortingByValue' and tag appropriate Cartesian traces

Author: antoinerg

src/traces/carpet/index.js

@@ -19,7 +19,7 @@ module.exports = {
     moduleType: 'trace',
     name: 'carpet',
     basePlotModule: require('../../plots/cartesian'),
-    categories: ['cartesian', 'svg', 'carpet', 'carpetAxis', 'notLegendIsolatable', 'noMultiCategory', 'noHover'],
+    categories: ['cartesian', 'svg', 'carpet', 'carpetAxis', 'notLegendIsolatable', 'noMultiCategory', 'noHover', 'noSortingByValue'],
     meta: {
         description: [
             'The data describing carpet axis layout is set in `y` and (optionally)',

src/traces/contourcarpet/index.js

@@ -19,7 +19,7 @@ module.exports = {
     moduleType: 'trace',
     name: 'contourcarpet',
     basePlotModule: require('../../plots/cartesian'),
-    categories: ['cartesian', 'svg', 'carpet', 'contour', 'symbols', 'showLegend', 'hasLines', 'carpetDependent', 'noHover'],
+    categories: ['cartesian', 'svg', 'carpet', 'contour', 'symbols', 'showLegend', 'hasLines', 'carpetDependent', 'noHover', 'noSortingByValue'],
     meta: {
         hrName: 'contour_carpet',
         description: [

src/traces/image/index.js

@@ -20,7 +20,7 @@ module.exports = {
     moduleType: 'trace',
     name: 'image',
     basePlotModule: require('../../plots/cartesian'),
-    categories: ['cartesian', 'svg', '2dMap'],
+    categories: ['cartesian', 'svg', '2dMap', 'noSortingByValue'],
     animatable: false,
     meta: {
         description: [

test/jasmine/tests/calcdata_test.js

@@ -885,11 +885,9 @@ describe('calculated data and points', function() {
                 return t.categories.length && t.categories.indexOf('cartesian') !== -1;
             });
 
-            // excludedTraces are traces that do not support sorting by value
-            var excludedTraces = [ 'carpet', 'contourcarpet', 'image'];
-
+            // exclude traces that do not support sorting by value
             var supportedCartesianTraces = cartesianTraces.filter(function(t) {
-                if(excludedTraces.indexOf(t.type) === -1) return true;
+                if(t.categories.indexOf('noSortingByValue') === -1) return true;
             });
 
             var cat = ['a', 'b', 'c'];