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);
 }

src/traces/scatterquiver/hover.js

@@ -17,20 +17,20 @@ module.exports = function hoverPoints(pointData, xval, yval, hovermode) {
     var closestPoint = null;
     var closestIndex = -1;
 
-    // Check each arrow segment
+    // Check each arrow point
     for(var i = 0; i < cd.length; i++) {
-        var segment = cd[i];
-        if(segment.length < 2) continue;
+        var point = cd[i];
+        if(!point || point.x === undefined || point.y === undefined) continue;
 
         // Calculate distance to the start point of the arrow
-        var x1 = xa.c2p(segment[0].x);
-        var y1 = ya.c2p(segment[0].y);
+        var x1 = xa.c2p(point.x);
+        var y1 = ya.c2p(point.y);
 
         var distance = Math.sqrt((xpx - x1) * (xpx - x1) + (ypx - y1) * (ypx - y1));
 
         if(distance < minDistance) {
             minDistance = distance;
-            closestPoint = segment[0]; // Use the start point for hover data
+            closestPoint = point; // Use the point for hover data
             closestIndex = i;
         }
     }
@@ -41,17 +41,17 @@ module.exports = function hoverPoints(pointData, xval, yval, hovermode) {
     var hoverPoint = {
         x: closestPoint.x,
         y: closestPoint.y,
-        u: trace.u[closestIndex],
-        v: trace.v[closestIndex],
+        u: closestPoint.u,
+        v: closestPoint.v,
         text: trace.text ? trace.text[closestIndex] : '',
         name: trace.name || '',
         trace: trace,
         index: closestIndex,
         // Set label values for proper hover formatting
         xLabelVal: closestPoint.x,
         yLabelVal: closestPoint.y,
-        uLabelVal: trace.u[closestIndex],
-        vLabelVal: trace.v[closestIndex],
+        uLabelVal: closestPoint.u,
+        vLabelVal: closestPoint.v,
         // Add spikeline support
         xa: pointData.xa,
         ya: pointData.ya,

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) {