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

fix iOS view sync regression#1904

Merged
brunoabinader merged 2 commits into
masterfrom
1904-nudge-transitions
Jul 20, 2015
Merged

fix iOS view sync regression#1904
brunoabinader merged 2 commits into
masterfrom
1904-nudge-transitions

Conversation

@brunoabinader
Copy link
Copy Markdown
Contributor

#1903 introduced a regression in the improvements from #1125. The addition of nudgeTransitions was not related to #1548, but rather happened in b838816 to allow iOS to insert position updates to native views between the GL frame draw and any changes to the Transform, so that the views were in sync with the GL frame.

@brunoabinader

@incanus incanus added bug iOS Mapbox Maps SDK for iOS labels Jul 20, 2015
@incanus
Copy link
Copy Markdown
Contributor Author

incanus commented Jul 20, 2015

More background: #1813

@brunoabinader
Copy link
Copy Markdown
Contributor

Sorry for that, I am reverting my revert and adding support for GLFW port. Thank you @incanus!

@brunoabinader
Copy link
Copy Markdown
Contributor

But perhaps that's more obfuscated.

👍 on that. The code in there explicitly creates the dirty variable which could also be removed, but most probably left there for better readability.

@brunoabinader brunoabinader merged commit 0b8cd02 into master Jul 20, 2015
@brunoabinader brunoabinader deleted the 1904-nudge-transitions branch July 20, 2015 19:17
@incanus
Copy link
Copy Markdown
Contributor Author

incanus commented Jul 20, 2015

👍

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

Labels

bug iOS Mapbox Maps SDK for iOS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants