Use isArrayOrTypedArray

Author: degzhaus

src/traces/quiver/attributes.js

@@ -254,5 +254,3 @@ attrs.hovertemplate = extendFlat({}, hovertemplateAttrs({}, {
 }));
 
 module.exports = attrs;
-
-

src/traces/quiver/calc.js

@@ -29,7 +29,7 @@ module.exports = function calc(gd, trace) {
     var normMax = -Infinity;
     var cMin = Infinity;
     var cMax = -Infinity;
-    var hasC = Array.isArray(trace.c);
+    var hasC = Lib.isArrayOrTypedArray(trace.c);
 
     for(var i = 0; i < len; i++) {
         var cdi = cd[i] = { i: i };
@@ -75,5 +75,3 @@ module.exports = function calc(gd, trace) {
 
     return cd;
 };
-
-

src/traces/quiver/defaults.js

@@ -19,19 +19,20 @@ module.exports = function supplyDefaults(traceIn, traceOut, defaultColor, layout
     var v = coerce('v');
 
     // Simple validation - check if we have the required arrays
-    if(!x || !Array.isArray(x) || x.length === 0 ||
-       !y || !Array.isArray(y) || y.length === 0) {
+    // Use Lib.isArrayOrTypedArray to support both regular arrays and typed arrays
+    if(!x || !Lib.isArrayOrTypedArray(x) || x.length === 0 ||
+       !y || !Lib.isArrayOrTypedArray(y) || y.length === 0) {
         traceOut.visible = false;
         return;
     }
 
     // If u/v are missing, default to zeros so the trace participates in calc/category logic
     var len = Math.min(x.length, y.length);
-    if(!Array.isArray(u) || u.length === 0) {
+    if(!Lib.isArrayOrTypedArray(u) || u.length === 0) {
         traceOut.u = new Array(len);
         for(var i = 0; i < len; i++) traceOut.u[i] = 0;
     }
-    if(!Array.isArray(v) || v.length === 0) {
+    if(!Lib.isArrayOrTypedArray(v) || v.length === 0) {
         traceOut.v = new Array(len);
         for(var j = 0; j < len; j++) traceOut.v[j] = 0;
     }
@@ -86,5 +87,3 @@ module.exports = function supplyDefaults(traceIn, traceOut, defaultColor, layout
     // Set the data length
     traceOut._length = len;
 };
-
-

src/traces/quiver/event_data.js

@@ -8,5 +8,3 @@ module.exports = function eventData(out, pt, trace, cd, pointNumber) {
     out.pointNumber = pointNumber;
     out.trace = trace;
 };
-
-

src/traces/quiver/format_labels.js

@@ -0,0 +1,25 @@
+'use strict';
+
+var Axes = require('../../plots/cartesian/axes');
+
+module.exports = function formatLabels(cdi, trace, fullLayout) {
+    var labels = {};
+
+    var xa = Axes.getFromId({ _fullLayout: fullLayout }, trace.xaxis || 'x');
+    var ya = Axes.getFromId({ _fullLayout: fullLayout }, trace.yaxis || 'y');
+
+    var x = cdi.x;
+    var y = cdi.y;
+
+    labels.xLabel = Axes.tickText(xa, xa.c2l(x), true).text;
+    labels.yLabel = Axes.tickText(ya, ya.c2l(y), true).text;
+
+    var u = trace.u ? trace.u[cdi.i] : 0;
+    var v = trace.v ? trace.v[cdi.i] : 0;
+
+    // Format u and v as plain numbers
+    labels.uLabel = String(u);
+    labels.vLabel = String(v);
+
+    return labels;
+};

src/traces/quiver/hover.js

@@ -73,5 +73,3 @@ module.exports = function hoverPoints(pointData, xval, yval, hovermode) {
 
     return [hoverPoint];
 };
-
-

src/traces/quiver/index.js

@@ -29,5 +29,3 @@ module.exports = {
         ].join(' ')
     }
 };
-
-

src/traces/quiver/plot.js

@@ -191,7 +191,7 @@ function plotOne(gd, idx, plotinfo, cdscatter, cdscatterAll, element, transition
         lineSegments.style('stroke', function(cdi) {
             var cArr = trace.c;
             var value;
-            if (Array.isArray(cArr) && cArr.length > cdi.i && isFinite(cArr[cdi.i])) {
+            if (Lib.isArrayOrTypedArray(cArr) && cArr.length > cdi.i && isFinite(cArr[cdi.i])) {
                 value = cArr[cdi.i];
             } else {
                 var uVal = (trace.u && trace.u[cdi.i]) || 0;
@@ -212,5 +212,3 @@ function plotOne(gd, idx, plotinfo, cdscatter, cdscatterAll, element, transition
             .style('opacity', 1);
     }
 }
-
-

src/traces/quiver/select_points.js

@@ -37,5 +37,3 @@ module.exports = function selectPoints(searchInfo, selectionTester) {
 
     return selection;
 };
-
-

src/traces/quiver/style.js

@@ -14,5 +14,3 @@ module.exports = function style(gd, calcTrace) {
     s.selectAll('path.js-line')
         .call(Drawing.lineGroupStyle, trace.line || {});
 };
-
-