Ticket #2075 (closed bug: fixed)

Opened 4 years ago

Last modified 4 years ago

google layer issues in late rendered maps

Reported by: pgiraud Owned by: euzuro
Priority: critical Milestone: 2.8 Release
Component: Layer.Google Version: 2.8 RC2
Keywords: Cc:
State: Complete

Description

I tried to adapt the late-render.html example to make it work with a google layer, but the tiles are not correctly displayed (only half of the map is shown).

var map, layer;
        function init(){
            var options = {
                projection: new OpenLayers.Projection("EPSG:900913"),
                displayProjection: new OpenLayers.Projection("EPSG:4326"),
                units: "m",
                maxResolution: 156543.0339,
                maxExtent: new OpenLayers.Bounds(-20037508.34, -20037508.34,
                                                 20037508.34, 20037508.34)
            };
            map = new OpenLayers.Map(options);

            layer = new OpenLayers.Layer.Google("Google", {sphericalMercator:true});

            map.addLayer(layer);
            map.render("container_id");
            map.zoomToExtent(
                new OpenLayers.Bounds(
                    68.774414, 11.381836, 123.662109, 34.628906
                ).transform(map.displayProjection, map.projection)
            );
        }

Attachments

late-google.patch Download (1.2 KB) - added by ahocevar 4 years ago.
late-render.html Download (2.1 KB) - added by ahocevar 4 years ago.
modified example to see the issue and test the patch

Change History

Changed 4 years ago by pgiraud

  • owner set to euzuro
  • priority changed from minor to critical
  • component changed from Map to Layer.Google
  • milestone changed from 2.9 Release to 2.8 Release

Thanks to Eric who helped me, reverting to revision r9333 fixes the issue, which seems to demonstrate that there's a regression on that point with the patch committed for #1797.

Sorry about that but this has to be considered as blocker for 2.8.

Changed 4 years ago by ahocevar

  • state set to Review

The above patch fixes the issue and can be tested with the modified late-render.html example (don't forget to include the Google Maps script tag).

Changed 4 years ago by ahocevar

Changed 4 years ago by ahocevar

modified example to see the issue and test the patch

Changed 4 years ago by pgiraud

  • state changed from Review to Commit

Indeed, this looks good (as always). Thanks Andreas. Please commit.

Changed 4 years ago by ahocevar

  • keywords pullup added
  • state changed from Commit to Pullup

(In [9362]) use event-based checkResize not only for invisible layers, but also for layers that have not finished loading. r=pgiraud (pullup #2075)

Changed 4 years ago by crschmidt

  • keywords pullup removed
  • status changed from new to closed
  • state changed from Pullup to Complete
  • resolution set to fixed

(In [9406]) Pullups for OL 2.8 RC3.

jQuery lib fix (Closes #1391) getRenderedSize regression (Closes #1906) element.scrolls error with panzoombar (Closes #2054) createUrlObject bug (Closes #2060) google layer in late rendered maps (Closes #2075) IE6/Lang.nb bug (Closes #2093) Layer.TMS/TileCache bugs (Closes #2099) (Closes #2100) Graphic names issues (Closes #2101)

Note: See TracTickets for help on using tickets.