feels weird

Author: degzhaus

src/plots/plots.js

@@ -3181,6 +3181,7 @@ function sortAxisCategoriesByValue(axList, gd) {
                         value = cdi.s;
                         if(value === undefined) value = cdi.v;
                         if(value === undefined) value = isX ? cdi.y : cdi.x;
+                        
 
                         if(!Array.isArray(value)) {
                             if(value === undefined) value = [];

src/traces/scatterquiver/calc.js

@@ -45,11 +45,12 @@ module.exports = function calc(gd, trace) {
 
             // For category aggregation, we need to set the right properties
             // The category aggregation looks for:
-            // - catIndex = cdi.p (position) or cdi[axLetter] (x/y coordinate)
+            // - catIndex = cdi.p (position) or cdi[axLetter] (x/y coordinate)  
             // - value = cdi.s (size) or cdi.v (value) or opposite coordinate
-            // We'll let the axis system handle the category index mapping
-            cdi.s = Math.sqrt(u[i] * u[i] + v[i] * v[i]); // magnitude as size/value
-            cdi.v = cdi.s; // also set v for value-based aggregation
+            // For scatterquiver, use u component as the value for category ordering
+            cdi.s = u[i]; // use u component for size/value
+            cdi.v = u[i]; // use u component for value-based aggregation
+            
 
             // Calculate arrow components for rendering
             var dx = u[i] * scale * (scaleRatio || 1);
@@ -100,6 +101,9 @@ function isNumeric(val) {
 
 function calcAxisExpansion(gd, trace, xa, ya, x, y) {
     var serieslen = trace._length;
+    var fullLayout = gd._fullLayout;
+    var xId = xa._id;
+    var yId = ya._id;
     var ppad = 0.1; // padding for arrows
 
     // Calculate padding based on arrow lengths
@@ -112,6 +116,11 @@ function calcAxisExpansion(gd, trace, xa, ya, x, y) {
         }
     }
 
-    xa.expand(x, {ppad: ppad});
-    ya.expand(y, {ppad: ppad});
+    // Use the same approach as scatter trace
+    var xOptions = {padded: true, ppad: ppad};
+    var yOptions = {padded: true, ppad: ppad};
+    
+    // N.B. asymmetric splom traces call this with blank {} xa or ya
+    if(xId) trace._extremes[xId] = Axes.findExtremes(xa, x, xOptions);
+    if(yId) trace._extremes[yId] = Axes.findExtremes(ya, y, yOptions);
 }

test/jasmine/tests/calcdata_test.js

@@ -998,7 +998,7 @@ describe('calculated data and points', function() {
                     }
                 }
 
-                return Lib.extendDeep({}, {
+                var baseData = {
                     orientation: axName === 'yaxis' ? 'h' : 'v',
                     type: type,
                     x: cat,
@@ -1032,7 +1032,15 @@ describe('calculated data and points', function() {
                             values: b
                         }
                     ]
-                });
+                };
+
+                // For scatterquiver, add u and v vector components
+                if(type === 'scatterquiver') {
+                    baseData.u = data;
+                    baseData.v = data;
+                }
+
+                return Lib.extendDeep({}, baseData);
             }
 
             supportedCartesianTraces.forEach(function(trace) {