Ticket #1157: setextent.2.patch

lib/OpenLayers/Renderer/SVG.js

@@ -121 +121 @@
-        // Set the viewbox -- the left/top will be pixels-dragged-since-res change,
-        // the width/height will be pixels.
-        var extentString = left + " " + top + " " + 
-extent.getWidth() / resolution + " " + extent.getHeight() / resolution
+this.size.w + " " + this.size.h
-        //var extentString = extent.left / resolution + " " + -extent.top / resolution + " " + 
-        this.rendererRoot.setAttributeNS(null, "viewBox", extentString);
-    },

lib/OpenLayers/Renderer/VML.js

@@ -79 +79 @@
-        var resolution = this.getResolution();
-    
-        var org = extent.left/resolution + " " + 
-extent.top/resolution
+(extent.top/resolution - this.size.h)
-        this.root.setAttribute("coordorigin", org);
-
-
-
+
-        this.root.setAttribute("coordsize", size);
+        
+        // flip the VML display Y axis upside down so it 
+        // matches the display Y axis of the map
+        this.root.style.flip = "y";
-    },
-
-

tests/Renderer/SVG.html

@@ -59 +59 @@
-        }
-                
-        var r = new OpenLayers.Renderer.SVG(document.body);
+        r.setSize(new OpenLayers.Size(4,4));
-        r.map = {
-            getResolution: function() {
-                return 0.5;
@@ -78 +79 @@
-        // test extent changes
-        var extent = new OpenLayers.Bounds(4,3,2,1);
-        r.setExtent(extent);
--4 -
+4 
-
-        OpenLayers.Renderer.Elements.prototype.setExtent =
-            OpenLayers.Renderer.Elements.prototype._setExtent;

tests/Renderer/VML.html

@@ -59 +59 @@
-        }
-                
-        var r = new OpenLayers.Renderer.VML(document.body);
+        r.setSize(new OpenLayers.Size(4,4));
-        r.map = {
-            getResolution: function() {
-                return 0.5;
@@ -70 +71 @@
-        
-        t.eq(g_SetExtent, true, "Elements.setExtent() called");
-        
-8
--
+4
+
-        
-        OpenLayers.Renderer.Elements.prototype.setExtent =
-            OpenLayers.Renderer.Elements.prototype._setExtent;