Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

fix symbol flickering across antimeridian #11938

Merged
merged 2 commits into from
May 22, 2018
Merged

fix symbol flickering across antimeridian #11938

merged 2 commits into from
May 22, 2018

Conversation

ansis
Copy link
Contributor

@ansis ansis commented May 17, 2018

fix #11878

This fixes:

  • symbol flickering when crossing the antimeridian
  • tile flickering when crossing the antimerdian

I pushed the glfw-app changes I used to test the flickering to port-antimeridan-fix-debug

@ChrisLoer

@ansis ansis requested a review from ChrisLoer May 17, 2018 19:02
Copy link
Contributor

@ChrisLoer ChrisLoer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! I did some quick manual testing with GLFW and couldn't reproduce any of the previous antimeridian issues.

for (auto& zoomIndex : indexes) {
std::map<OverscaledTileID,TileLayerIndex> newZoomIndex;
for (auto& index : zoomIndex.second) {
// change the tileID's wrap and it to a new index
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: Looks like this should be something like "and move its index"?

@ChrisLoer
Copy link
Contributor

Note: I had to restart the ios-sanitize and android-debug-arm-v7 tests to get them to pass, but it looked like it was just some sort of unrelated CI timeout.

@ansis ansis merged commit ba07c87 into master May 22, 2018
@friedbunny friedbunny added Core The cross-platform C++ core, aka mbgl needs changelog Indicates PR needs a changelog entry prior to merging. labels May 22, 2018
@friedbunny
Copy link
Contributor

Added the needs changelog label here, as this should have platform changelog entries added at some point.

@friedbunny friedbunny deleted the port-antimeridian-fix branch May 22, 2018 19:27
@friedbunny
Copy link
Contributor

iOS changelog added in #12086.

@friedbunny friedbunny removed the needs changelog Indicates PR needs a changelog entry prior to merging. label Jun 11, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Core The cross-platform C++ core, aka mbgl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Symbols flicker on crossing antimeridian
3 participants