revise defaults

Author: archmoj

src/plots/mapbox/layout_attributes.js

@@ -97,23 +97,31 @@ var attrs = module.exports = overrideAll({
 
     westbound: {
         valType: 'number',
-        dflt: null,
-        description: 'Sets the minimum longitude of the map (in degrees East).'
+        description: [
+            'Sets the minimum longitude of the map (in degrees East)',
+            'if `eastbound`, `southbound` and `northbound` are declared.'
+        ].join(' ')
     },
     eastbound: {
         valType: 'number',
-        dflt: null,
-        description: 'Sets the maximum longitude of the map (in degrees East).'
+        description: [
+            'Sets the maximum longitude of the map (in degrees East)',
+            'if `westbound`, `southbound` and `northbound` are declared.'
+        ].join(' ')
     },
     southbound: {
         valType: 'number',
-        dflt: null,
-        description: 'Sets the minimum latitude of the map (in degrees North).'
+        description: [
+            'Sets the minimum latitude of the map (in degrees North)',
+            'if `eastbound`, `westbound` and `northbound` are declared.'
+        ].join(' ')
     },
     northbound: {
         valType: 'number',
-        dflt: null,
-        description: 'Sets the maximum latitude of the map (in degrees North).'
+        description: [
+            'Sets the maximum latitude of the map (in degrees North)',
+            'if `eastbound`, `westbound` and `southbound` are declared.'
+        ].join(' ')
     },
 
     layers: templatedArray('layer', {

src/plots/mapbox/layout_defaults.js

@@ -26,10 +26,21 @@ function handleDefaults(containerIn, containerOut, coerce, opts) {
     coerce('bearing');
     coerce('pitch');
 
-    coerce('westbound');
-    coerce('eastbound');
-    coerce('southbound');
-    coerce('northbound');
+    var westbound = coerce('westbound');
+    var eastbound = coerce('eastbound');
+    var southbound = coerce('southbound');
+    var northbound = coerce('northbound');
+    if(
+        westbound === undefined ||
+        eastbound === undefined ||
+        southbound === undefined ||
+        northbound === undefined
+    ) {
+        delete containerOut.westbound;
+        delete containerOut.eastbound;
+        delete containerOut.southbound;
+        delete containerOut.northbound;
+    }
 
     handleArrayContainerDefaults(containerIn, containerOut, {
         name: 'layers',

src/plots/mapbox/mapbox.js

@@ -96,9 +96,11 @@ proto.createMap = function(calcData, fullLayout, resolve, reject) {
     var southbound = opts.southbound;
     var northbound = opts.northbound;
     var maxBounds = (
+        /*
         westbound !== undefined &&
         eastbound !== undefined &&
         southbound !== undefined &&
+        */
         northbound !== undefined
     ) ? [[westbound, southbound], [eastbound, northbound]] : null;
 

test/plot-schema.json

@@ -3087,8 +3087,7 @@
      }
     },
     "eastbound": {
-     "description": "Sets the maximum longitude of the map (in degrees East).",
-     "dflt": null,
+     "description": "Sets the maximum longitude of the map (in degrees East) if `westbound`, `southbound` and `northbound` are declared.",
      "editType": "plot",
      "valType": "number"
     },
@@ -3313,8 +3312,7 @@
      "role": "object"
     },
     "northbound": {
-     "description": "Sets the maximum latitude of the map (in degrees North).",
-     "dflt": null,
+     "description": "Sets the maximum latitude of the map (in degrees North) if `eastbound`, `westbound` and `southbound` are declared.",
      "editType": "plot",
      "valType": "number"
     },
@@ -3326,8 +3324,7 @@
     },
     "role": "object",
     "southbound": {
-     "description": "Sets the minimum latitude of the map (in degrees North).",
-     "dflt": null,
+     "description": "Sets the minimum latitude of the map (in degrees North) if `eastbound`, `westbound` and `northbound` are declared.",
      "editType": "plot",
      "valType": "number"
     },
@@ -3359,8 +3356,7 @@
      "valType": "any"
     },
     "westbound": {
-     "description": "Sets the minimum longitude of the map (in degrees East).",
-     "dflt": null,
+     "description": "Sets the minimum longitude of the map (in degrees East) if `eastbound`, `southbound` and `northbound` are declared.",
      "editType": "plot",
      "valType": "number"
     },