Add special handling for Albers USA

Author: camdecoster

src/plots/geo/geo.js

@@ -353,9 +353,12 @@ proto.updateProjection = function (geoCalcData, fullLayout) {
     // so here's this hack to make it respond to 'geoLayout.center'
     if (geoLayout._isAlbersUsa) {
         var centerPx = projection([center.lon, center.lat]);
-        var tt = projection.translate();
-
-        projection.translate([tt[0] - (centerPx[0] - tt[0]), tt[1] - (centerPx[1] - tt[1])]);
+        // If center isn't within the Albers USA bounds (clipped to the USA),
+        // `projection(...)` returns null so skip the recentering
+        if (centerPx) {
+            var tt = projection.translate();
+            projection.translate([tt[0] - (centerPx[0] - tt[0]), tt[1] - (centerPx[1] - tt[1])]);
+        }
     }
 };
 

src/plots/geo/layout_defaults.js

@@ -201,8 +201,8 @@ function handleGeoDefaults(geoLayoutIn, geoLayoutOut, coerce, opts) {
 
     // Only use fitbounds if user hasn't set any view attributes. This prevents
     // user-specified view info from being ignored.
-    const fitbounds = coerce('fitbounds');
-    if (fitbounds) {
+    coerce('fitbounds');
+    if (geoLayoutOut.fitbounds) {
         const centerIn = geoLayoutIn.center || {};
         const projectionIn = geoLayoutIn.projection || {};
         const rotationIn = projectionIn.rotation || {};
@@ -217,7 +217,8 @@ function handleGeoDefaults(geoLayoutIn, geoLayoutOut, coerce, opts) {
             lonaxisIn.range,
             lataxisIn.range
         ].some((d) => d !== undefined);
-        if (hasUserView) geoLayoutOut.fitbounds = false;
+        // The Albers projection doesn't need a fit, so skip here
+        if (hasUserView || isAlbersUsa) geoLayoutOut.fitbounds = false;
     }
 
     // Set auto-filled view attributes to null so updateProjection can