Merge pull request #3270 from elemoine/dragpan-false

Make ol.Overlay autoPan default to false

examples/measure.js

@@ -204,8 +204,7 @@ function createHelpTooltip() {
   helpTooltip = new ol.Overlay({
     element: helpTooltipElement,
     offset: [15, 0],
-    positioning: 'center-left',
+    positioning: 'center-left'
-    autoPan: false
   });
   map.addOverlay(helpTooltip);
 }
@@ -223,8 +222,7 @@ function createMeasureTooltip() {
   measureTooltip = new ol.Overlay({
     element: measureTooltipElement,
     offset: [0, -15],
-    positioning: 'bottom-center',
+    positioning: 'bottom-center'
-    autoPan: false
   });
   map.addOverlay(measureTooltip);
 }

examples/popup.js

@@ -31,6 +31,7 @@ closer.onclick = function() {
  */
 var overlay = new ol.Overlay(/** @type {olx.OverlayOptions} */ ({
   element: container,
+  autoPan: true,
   autoPanAnimation: {
     duration: 250
   }

examples/tileutfgrid.js

@@ -36,8 +36,7 @@ var nameElement = document.getElementById('country-name');
 var infoOverlay = new ol.Overlay({
   element: infoElement,
   offset: [15, 15],
-  stopEvent: false,
+  stopEvent: false
-  autoPan: false
 });
 map.addOverlay(infoOverlay);
 

externs/olx.js

@@ -382,7 +382,7 @@ olx.OverlayOptions.prototype.insertFirst;
 /**
  * If set to `true` the map is panned when calling `setPosition`, so that the
  * overlay is entirely visible in the current viewport.
- * The default is `true`.
+ * The default is `false`.
  * @type {boolean|undefined}
  * @api
  */

src/ol/overlay.js

@@ -97,7 +97,7 @@ ol.Overlay = function(options) {
    * @private
    * @type {boolean}
    */
-  this.autoPan_ = goog.isDef(options.autoPan) ? options.autoPan : true;
+  this.autoPan_ = goog.isDef(options.autoPan) ? options.autoPan : false;
 
   /**
    * @private