New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zoomslider can become out-of sync with actual zoom #205

Closed
marcjansen opened this Issue Oct 29, 2013 · 4 comments

Comments

Projects
None yet
4 participants
@marcjansen
Member

marcjansen commented Oct 29, 2013

To reproduce:

Most of the time you will now see that the lower slider says the zoom is e.g. 7, while the slider in the map (and the map itself) will report the zoom to be 0 (GeoExt.panel.Map.guess().map.zoom === 0).

zoomslider-out-of-sync

There are two issues here:

  1. the sliders should never be out of sync
  2. the map should display the layer at zoomlevel 0 and not show a pink tile
@weskamm

This comment has been minimized.

Show comment
Hide comment
@weskamm

weskamm Nov 27, 2013

Contributor

I think this is related to a bug in OpenLayers, which has been introduced since Version 2.13.
Multiple calls of map.zoomTo() in a row will lead to an incorrect zoom of the map in the end. This was working fine in OL 2.12 but since 2.13 and in 2.13.1 it seems to be broken somehow. I already reported this on the dev Mailinglist of OpenLayers and think we should wait for responses to see if this problem can be fixed in OpenLayers.

Contributor

weskamm commented Nov 27, 2013

I think this is related to a bug in OpenLayers, which has been introduced since Version 2.13.
Multiple calls of map.zoomTo() in a row will lead to an incorrect zoom of the map in the end. This was working fine in OL 2.12 but since 2.13 and in 2.13.1 it seems to be broken somehow. I already reported this on the dev Mailinglist of OpenLayers and think we should wait for responses to see if this problem can be fixed in OpenLayers.

@bartvde

This comment has been minimized.

Show comment
Hide comment
@bartvde

bartvde Nov 27, 2013

Member

@weskamm did you try an OL 2.13 map without a tileManager?

Member

bartvde commented Nov 27, 2013

@weskamm did you try an OL 2.13 map without a tileManager?

@weskamm

This comment has been minimized.

Show comment
Hide comment
@weskamm

weskamm Nov 27, 2013

Contributor

@bartvde that doesnt solve the problem, but when setting the zoomMethod of the map to null, like described on the mailinglist, the problem is gone

Contributor

weskamm commented Nov 27, 2013

@bartvde that doesnt solve the problem, but when setting the zoomMethod of the map to null, like described on the mailinglist, the problem is gone

@ahocevar

This comment has been minimized.

Show comment
Hide comment
@ahocevar

ahocevar Nov 27, 2013

Member

This does not have anything to do with the TileManager. It is because
during animated zooming, you cannot rely on the value you get for
map.getZoom(). However, if you register for a zoomend listener, you can
rely on the getZoom() value in that handler.

Andreas.

On Wed, Nov 27, 2013 at 11:57 AM, Johannes Weskamm <notifications@github.com

wrote:

@bartvde https://github.com/bartvde that doesnt solve the problem, but
when setting the zoomMethod of the map to null, like described on the
mailinglist, the problem is gone


Reply to this email directly or view it on GitHubhttps://github.com/geoext/geoext2/issues/205#issuecomment-29375824
.

Member

ahocevar commented Nov 27, 2013

This does not have anything to do with the TileManager. It is because
during animated zooming, you cannot rely on the value you get for
map.getZoom(). However, if you register for a zoomend listener, you can
rely on the getZoom() value in that handler.

Andreas.

On Wed, Nov 27, 2013 at 11:57 AM, Johannes Weskamm <notifications@github.com

wrote:

@bartvde https://github.com/bartvde that doesnt solve the problem, but
when setting the zoomMethod of the map to null, like described on the
mailinglist, the problem is gone


Reply to this email directly or view it on GitHubhttps://github.com/geoext/geoext2/issues/205#issuecomment-29375824
.

weskamm added a commit to weskamm/geoext2 that referenced this issue Dec 20, 2013

fixing issue #205 (Zoomslider can become out-of sync with actual zoom…
…) by setting zoomMethod to null.

Restricting the maps iresolution to match layers requirements

weskamm added a commit that referenced this issue Jan 6, 2014

Changed doc comments for issue #205
Added note to address issue #205, fixed the example code

weskamm added a commit that referenced this issue Jan 6, 2014

Merge pull request #225 from weskamm/zoomfix
fixing issue #205 (Zoomslider can become out-of sync with actual zoom)

@weskamm weskamm closed this Jan 6, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment