Skip to content
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

Tiny map shake while zooming in on raster tiles #6150

Closed
ChrisLoer opened this issue Feb 13, 2018 · 4 comments · Fixed by #7426
Closed

Tiny map shake while zooming in on raster tiles #6150

ChrisLoer opened this issue Feb 13, 2018 · 4 comments · Fixed by #7426

Comments

@ChrisLoer
Copy link
Contributor

In v0.44 on the "raster tile source" example, if you zoom in you can notice (at least on Mac Chrome/Firefox/Safari) a slight jitter back and forth:

raster shake on zoom

My initial guess is that this has something to do with the recent change to align raster tile projection matrices to a pixel grid.

/cc @anandthakker @kkaefer @mollymerp

@ChrisLoer ChrisLoer changed the title Tiny map shaking while zooming in on raster tiles Tiny map shake while zooming in on raster tiles Feb 13, 2018
@jfirebaugh
Copy link
Contributor

See #6709 and mapbox/mapbox-gl-native#11737 for similar reports.

@kkaefer What do you make of these issues? Is it possible to reconcile the simultaneous expectation of crisp raster imagery and zero "stutter" between a raster overlay and vector basemap?

@P-Zenker
Copy link

Any updates to this issue(s)?

@mourner
Copy link
Member

mourner commented Aug 2, 2018

@jfirebaugh maybe it should pixel-align when idle, and not align while animating?

@wboykinm
Copy link

@jfirebaugh I can confirm the effect is pretty crazy when slowly easing. Here's a raster tileset on top of a style that's meant to match, but you can see the foreground overlay dancing all over the base:

jitter

(Might be better-visible in the example map itself than in the above poor-framerate gif)

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

Successfully merging a pull request may close this issue.

5 participants