revise function and variable names

Author: archmoj

src/components/selections/select.js

@@ -50,13 +50,13 @@ var p2r = helpers.p2r;
 var axValue = helpers.axValue;
 var getTransform = helpers.getTransform;
 
-function cartesianDrag(dragOptions) {
+function hasSubplot(dragOptions) {
     // N.B. subplot may be falsy e.g zero sankey index!
     return dragOptions.subplot !== undefined;
 }
 
 function prepSelect(evt, startX, startY, dragOptions, mode) {
-    var isCartesianDrag = cartesianDrag(dragOptions);
+    var isCartesian = hasSubplot(dragOptions);
 
     var isFreeMode = freeMode(mode);
     var isRectMode = rectMode(mode);
@@ -71,7 +71,7 @@ function prepSelect(evt, startX, startY, dragOptions, mode) {
     var gd = dragOptions.gd;
     var fullLayout = gd._fullLayout;
     var immediateSelect = isSelectMode && fullLayout.newselection.mode === 'immediate' &&
-        !isCartesianDrag; // N.B. only cartesian subplots have persistent selection
+        !isCartesian; // N.B. only cartesian subplots have persistent selection
 
     var zoomLayer = fullLayout._zoomlayer;
     var dragBBox = dragOptions.element.getBoundingClientRect();
@@ -119,7 +119,7 @@ function prepSelect(evt, startX, startY, dragOptions, mode) {
             opacity: isDrawMode ? newStyle.opacity / 2 : 1,
             fill: (isDrawMode && !isOpenMode) ? newStyle.fillcolor : 'none',
             stroke: newStyle.line.color || (
-                isCartesianDrag ?
+                isCartesian ?
                     '#7f7f7f' : // non-cartesian subplot
                     Color.contrast(gd._fullLayout.plot_bgcolor) // cartesian subplot
             ),
@@ -152,7 +152,7 @@ function prepSelect(evt, startX, startY, dragOptions, mode) {
 
     if(immediateSelect && !evt.shiftKey) {
         dragOptions._clearSubplotSelections = function() {
-            if(isCartesianDrag) return;
+            if(isCartesian) return;
 
             var xRef = xAxis._id;
             var yRef = yAxis._id;
@@ -718,7 +718,7 @@ function clearSelectionsCache(dragOptions, immediateSelect) {
             var selections;
             if(
                 isSelectMode &&
-                !cartesianDrag(dragOptions) // only allow cartesian - no mapbox for now
+                !hasSubplot(dragOptions) // only allow cartesian - no mapbox for now
             ) {
                 selections = newSelections(outlines, dragOptions);
             }
@@ -758,7 +758,7 @@ function determineSearchTraces(gd, xAxes, yAxes, subplot) {
         if(trace.visible !== true || !trace._module || !trace._module.selectPoints) continue;
 
         if(
-            cartesianDrag({subplot: subplot}) &&
+            hasSubplot({subplot: subplot}) &&
             (trace.subplot === subplot || trace.geo === subplot)
         ) {
             searchTraces.push(createSearchInfo(trace._module, cd, xAxes[0], yAxes[0]));