Changeset 6371

Timestamp:

02/25/08 22:11:55 (11 months ago)

Author:

crschmidt

Message:

Add early version of transition patch, from #933

Files:

• sandbox/crschmidt/demo/examples/transition.html (added)
• sandbox/crschmidt/demo/lib/OpenLayers/Layer.js (modified) (1 diff)
• sandbox/crschmidt/demo/lib/OpenLayers/Tile.js (modified) (6 diffs)
• sandbox/crschmidt/demo/lib/OpenLayers/Tile/Image.js (modified) (5 diffs)

sandbox/crschmidt/demo/lib/OpenLayers/Layer.js

@@ -249 +249 @@
-    wrapDateLine: false,
-    
+    /**
+     * APIProperty: transitionEffect
+     * {String} The transition effect to use when the map is panned or
+     * zoomed.  There is currently two supported values:
+     * null - no transition effect (the default).
+     * keep - Keep tiles around. This uses double-buffering: this
+     *    type of transition effect is only useful for singleTile layers,
+     *    where the tile will be kept in place on drag.
+     * resize - existing tiles are resized on zoom to provide a visual
+     *   effect of the zoom having taken place immediately.  As the
+     *   new tiles become available, they are drawn over top of the
+     *    resized tiles.
+     */
+    transitionEffect: null,
-    
-    /**

sandbox/crschmidt/demo/lib/OpenLayers/Tile.js

@@ -81 +81 @@
-    isLoading: false,
-    
+    /**
+     * Property: isBackBuffer
+     * {Boolean} Is this tile a back buffer tile?
+     */
+    isBackBuffer: false,
+    
+    /**
+     * Property: isFirstDraw
+     * {Boolean} Is this the first time the tile is being drawn?
+     * This is used to force resetBackBuffer to synchronize
+     * the backBufferTile with the foreground tile the first time
+     * the foreground tile loads so that if the user zooms
+     * before the layer has fully loaded, the backBufferTile for
+     * tiles that have been loaded can be used.
+     */
+    isFirstDraw: true,
+        
+    /**
+     * Property: backBufferTile
+     * {<OpenLayers.Tile>} A clone of the tile used to create transition
+     * effects when the tile is moved or changes resolution.
+     */
+    backBufferTile: null,
+        
-    /** TBD 3.0 -- remove 'url' from the list of parameters to the constructor.
-     *             there is no need for the base tile class to have a url.
@@ -112 +136 @@
-     */
-    destroy:function() {
+        if (this.layer.transitionEffect) {
+            this.layer.events.unregister("loadend", this, this.resetBackBuffer);
+            this.events.unregister('loadend', this, this.resetBackBuffer);            
+        } else {
+            this.events.unregister('loadend', this, this.showTile);
+        }
-        this.layer  = null;
-        this.bounds = null;
@@ -119 +149 @@
-        this.events.destroy();
-        this.events = null;
+        
+        /* clean up the backBufferTile if it exists */
+        if (this.backBufferTile) {
+            this.backBufferTile.destroy();
+            this.backBufferTile = null;
+        }
-    },
-    
@@ -158 +194 @@
-    draw: function() {
-        
-        //clear tile's contents and mark as not drawn
-        this.clear();
-        
-        var maxExtent = this.layer.maxExtent;
-        var withinMaxExtent = (maxExtent &&
@@ -167 +200 @@
-        // The only case where we *wouldn't* want to draw the tile is if the 
-        // tile is outside its layer's maxExtent.
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-    },
-    
@@ -238 +314 @@
-        return bounds;
-    },        
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-    CLASS_NAME: "OpenLayers.Tile"
-});

sandbox/crschmidt/demo/lib/OpenLayers/Tile/Image.js

@@ -147 +147 @@
-        }
-        
+        return this.renderTile();
+    },
+    
+    /**
+     * Method: renderTile
+     *
+     * internal function to actually initialize the image tile,
+     * position it correctly, and set its url
+     */
+    renderTile: function() {
-        if (this.imgDiv == null) {
-            this.initImgDiv();
@@ -163 +173 @@
-                    null, null, imageSize, this.url);
-        } else {
-            this.imgDiv.src = this.url;
-            OpenLayers.Util.modifyDOMElement(this.imgDiv,
-                    null, null, imageSize) ;
+            this.imgDiv.src = this.url;
-        }
-        return true;
@@ -177 +187 @@
-    clear: function() {
-        if(this.imgDiv) {
-imgDiv.style.display = "none"
+hide()
-            if (OpenLayers.Tile.Image.useBlankTile) { 
-                this.imgDiv.src = OpenLayers.Util.getImagesLocation() + "blank.gif";
@@ -224 +234 @@
-            OpenLayers.Function.bind(this.checkImgURL, this) );
-        */
+        this.frame.style.zIndex = this.isBackBuffer ? 0 : 1;
-        this.frame.appendChild(this.imgDiv); 
-        this.layer.div.appendChild(this.frame); 
@@ -301 +312 @@
-            var loaded = this.layerAlphaHack ? this.imgDiv.firstChild.src : this.imgDiv.src;
-            if (!OpenLayers.Util.isEquivalentUrl(loaded, this.url)) {
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-    CLASS_NAME: "OpenLayers.Tile.Image"
-  }