Merge pull request #829 from bjornharrtell/fallThough

Default fallThrough to false

lib/OpenLayers/Map.js

@@ -384,9 +384,9 @@ OpenLayers.Map = OpenLayers.Class({
      * APIProperty: fallThrough
      * {Boolean} Should OpenLayers allow events on the map to fall through to
      *           other elements on the page, or should it swallow them? (#457)
-     *           Default is to fall through.
+     *           Default is to swallow.
      */
-    fallThrough: true,
+    fallThrough: false,
 
     /**
      * APIProperty: autoUpdateSize

notes/2.13.md

@@ -68,6 +68,19 @@ Corresponding issue/pull requests:
 
 * https://github.com/openlayers/openlayers/pull/423
 
+## Map property fallThrough defaults to false
+
+The behaviour controlled by map property fallThrough wasn't consistent and changes
+has been made to fix that. The change means that some events will fall through as
+was intended with fallThrough set to true. Defaulting fallThrough to false after
+this change is sensible in most situations and will probably be what most
+applications expect, but if you previously relied on events beeing passed through
+you will probably want to set fallThrough to true.
+
+Behavioural change was made in this commit:
+
+* https://github.com/openlayers/openlayers/commit/a6119f6a7528e013b922fd0d997a07df13f6bd6e
+
 # New Options for Build Script
 
 * add the contents of a file as a comment at the front of the build, for example, the output of 'git describe --tags' could be saved as a file and then included