OpenLayers OpenLayers

Ticket #480 (closed bug: fixed)

Opened 2 years ago

Last modified 2 years ago

Grid funkiness

Reported by: tschaub Assigned to: bwoodall
Priority: major Milestone: 2.3 Release
Component: general Version: 2.3 RC2
Keywords: Cc:
State:

Description (Last modified by euzuro)

Haven't looked into this, but it doesn't look right.

  1. open http://openlayers.org/viewer/?controls=panzoom,layerswitcher,mouse&layers=openlayers,gmaps
  2. switch to one of the Google layers
  3. switch back to the OpenLayers WMS
  4. see tiles outside the max extent from a different zoom level

Attachments

Tic480_and_grid_initTile (3.1 kB) - added by bwoodall on 02/14/07 07:47:36.
480-redo.patch.txt (3.2 kB) - added by crschmidt on 02/16/07 12:01:36.
improved patch
480-nocreateImage.patch (2.8 kB) - added by crschmidt on 02/16/07 12:18:09.
update of #480
480.patch (3.6 kB) - added by crschmidt on 02/16/07 12:21:28.
add layer 'state' variable
480.2.patch (4.7 kB) - added by euzuro on 02/16/07 14:34:47.
changed 'state' to 'viewRequestID', moved from layer to map, updated comments
480.3.patch (5.0 kB) - added by euzuro on 02/16/07 14:47:49.
moved assignment of .map to initImgDiv() function. also of note, this patch (and the one before it) makes a change to the html example in tests so that IE doesnt break
480.4.patch (4.7 kB) - added by euzuro on 02/16/07 15:04:52.
changed 'viewRequestID' from a unique, random string to a constantly increased integer

Change History

01/31/07 06:49:57 changed by crschmidt

  • priority changed from minor to major.
  • milestone changed from 2.4 Release to 2.3 Release.

I don't think we should push this off into 2.4, unfortuantely. We broke this once we were already in RC for 2.3, and unbreaking it now seemes like the right thing to do (even though I don't want to.)

02/06/07 21:58:40 changed by crschmidt

http://openlayers.org/pipermail/dev/2007-January/000258.html Temporary fix for 2.3

http://openlayers.org/pipermail/dev/2007-February/000273.html Real problem

02/14/07 07:47:36 changed by bwoodall

  • attachment Tic480_and_grid_initTile added.

02/14/07 07:55:59 changed by bwoodall

  • keywords set to review.
  • owner set to bwoodall.
  • status changed from new to assigned.

This fixes the problem when you zoom too fast (aka crschmidt grid_initTile ) and when you spin the mousewheel and the original problem that tschaub showed

Also passed "all tests" on linux FF

02/16/07 11:56:16 changed by euzuro

  • description changed.

02/16/07 12:01:36 changed by crschmidt

  • attachment 480-redo.patch.txt added.

improved patch

02/16/07 12:18:09 changed by crschmidt

  • attachment 480-nocreateImage.patch added.

update of #480

02/16/07 12:21:28 changed by crschmidt

  • attachment 480.patch added.

add layer 'state' variable

02/16/07 14:34:47 changed by euzuro

  • attachment 480.2.patch added.

changed 'state' to 'viewRequestID', moved from layer to map, updated comments

02/16/07 14:47:49 changed by euzuro

  • attachment 480.3.patch added.

moved assignment of .map to initImgDiv() function. also of note, this patch (and the one before it) makes a change to the html example in tests so that IE doesnt break

02/16/07 15:04:52 changed by euzuro

  • attachment 480.4.patch added.

changed 'viewRequestID' from a unique, random string to a constantly increased integer

02/16/07 15:15:49 changed by sderle

  • keywords changed from review to pullup.

patch 4 works for me. please apply.

02/16/07 15:25:31 changed by euzuro

applied to trunk with r2231

02/19/07 11:19:02 changed by euzuro

  • keywords deleted.
  • status changed from assigned to closed.
  • version set to 2.3 RC2.
  • resolution set to fixed.