Changed order of arguments to shiftPosition

Also avoided using String.includes.

Author: nicholas-esterer

src/components/annotations/draw.js

@@ -78,10 +78,10 @@ function drawOne(gd, index) {
 // the plot.
 // axDomainRef: if true and axa defined, draws relative to axis domain,
 // otherwise draws relative to data (if axa defined) or paper (if not).
-function shiftPosition(axa, options, dAx, axLetter, gs) {
+function shiftPosition(axa, dAx, axLetter, gs, options) {
     var optAx = options[axLetter];
     var axRef = options[axLetter + 'ref'];
-    var vertical = axLetter.includes('y');
+    var vertical = axLetter.indexOf('y') !== -1;
     var axDomainRef = Axes.getRefType(axRef) === 'domain';
     var gsDim = vertical ? gs.h : gs.w;
     if(axa) {
@@ -615,19 +615,19 @@ function drawRaw(gd, options, index, subplotId, xa, ya) {
                         annTextGroupInner.call(Drawing.setTranslate, xcenter, ycenter);
 
                         modifyItem('x',
-                            shiftPosition(xa, options, dx, 'x', gs));
+                            shiftPosition(xa, dx, 'x', gs, options));
                         modifyItem('y',
-                            shiftPosition(ya, options, dy, 'y', gs));
+                            shiftPosition(ya, dy, 'y', gs, options));
 
                         // for these 2 calls to shiftPosition, it is assumed xa, ya are
                         // defined, so gsDim will not be used, but we put it in
                         // anyways for consistency
                         if(options.axref === options.xref) {
-                            modifyItem('ax', shiftPosition(xa, options, dx, 'ax', gs));
+                            modifyItem('ax', shiftPosition(xa, dx, 'ax', gs, options));
                         }
 
                         if(options.ayref === options.yref) {
-                            modifyItem('ay', shiftPosition(ya, options, dy, 'ay', gs));
+                            modifyItem('ay', shiftPosition(ya, dy, 'ay', gs, options));
                         }
 
                         arrowGroup.attr('transform', 'translate(' + dx + ',' + dy + ')');
@@ -666,13 +666,13 @@ function drawRaw(gd, options, index, subplotId, xa, ya) {
                         // defined, so gsDim will not be used, but we put it in
                         // anyways for consistency
                         if(options.axref === options.xref) {
-                            modifyItem('ax', shiftPosition(xa, options, dx, 'ax', gs));
+                            modifyItem('ax', shiftPosition(xa, dx, 'ax', gs, options));
                         } else {
                             modifyItem('ax', options.ax + dx);
                         }
 
                         if(options.ayref === options.yref) {
-                            modifyItem('ay', shiftPosition(ya, options, dy, 'ay', gs.w));
+                            modifyItem('ay', shiftPosition(ya, dy, 'ay', gs.w, options));
                         } else {
                             modifyItem('ay', options.ay + dy);
                         }
@@ -683,7 +683,7 @@ function drawRaw(gd, options, index, subplotId, xa, ya) {
                         if(xa) {
                             // shiftPosition will not execute code where xa was
                             // undefined, so we use to calculate xUpdate too
-                            xUpdate = shiftPosition(xa, options, dx, 'x', gs);
+                            xUpdate = shiftPosition(xa, dx, 'x', gs, options);
                         } else {
                             var widthFraction = options._xsize / gs.w;
                             var xLeft = options.x + (options._xshift - options.xshift) / gs.w - widthFraction / 2;
@@ -695,7 +695,7 @@ function drawRaw(gd, options, index, subplotId, xa, ya) {
                         if(ya) {
                             // shiftPosition will not execute code where ya was
                             // undefined, so we use to calculate yUpdate too
-                            yUpdate = shiftPosition(ya, options, dy, 'y', gs);
+                            yUpdate = shiftPosition(ya, dy, 'y', gs, options);
                         } else {
                             var heightFraction = options._ysize / gs.h;
                             var yBottom = options.y - (options._yshift + options.yshift) / gs.h - heightFraction / 2;