[OpenLayers-Commits] r11465 - sandbox/ahocevar/layercontainer/lib/OpenLayers

[commits-20090109 at openlayers.org]

Author: erilem
Date: 2011-02-25 01:04:00 -0800 (Fri, 25 Feb 2011)
New Revision: 11465

Modified:
   sandbox/ahocevar/layercontainer/lib/OpenLayers/Map.js
Log:
API doc strings updates

Modified: sandbox/ahocevar/layercontainer/lib/OpenLayers/Map.js
===================================================================
--- sandbox/ahocevar/layercontainer/lib/OpenLayers/Map.js	2011-02-25 09:01:13 UTC (rev 11464)
+++ sandbox/ahocevar/layercontainer/lib/OpenLayers/Map.js	2011-02-25 09:04:00 UTC (rev 11465)
@@ -423,17 +423,18 @@
     
     /**
      * Property: minPx
-     * {<OpenLayers.Pixel>}
-     *
-     * lower left of maxExtent in viewport pixel space
+     * {<OpenLayers.Pixel>} Lower left of maxExtent in viewport pixel space.
+     *     Used to verify in moveByPx that the new location we're moving to
+     *     is valid. It is also used in the getLonLatFromViewPortPx function
+     *     of Layer.
      */
     minPx: null,
     
     /**
      * Property: maxPx
-     * {<OpenLayers.Pixel>}
-     *
-     * top right of maxExtent in viewport pixel space
+     * {<OpenLayers.Pixel>} Top right of maxExtent in viewport pixel space.
+     *     Used to verify in moveByPx that the new location we're moving to
+     *     is valid.
      */
     maxPx: null,
     
@@ -1686,8 +1687,6 @@
             var layer, i, len;
             for (i=0, len=this.layers.length; i<len; ++i) {
                 layer = this.layers[i];
-                // FIXME inRange must be taken into account somehow
-                // do we need to deal differently with the base layer?
                 if (layer.visibility) {
                     layer.moveByPx(dx, dy);
                     layer.events.triggerEvent("move");