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

Annotation view recycling re-enabled #2454

Merged

Conversation

alexcristici
Copy link
Collaborator

@alexcristici alexcristici commented May 30, 2024

Reverted code that disabled annotation view recycling, from this PR: #2148.
We don't need that workaround after fixing the presentsWithTransaction rendering in this PR: #2442.

Before presentsWithTransaction fix:

Before.MP4

After presentsWithTransaction fix:

After.MP4

@alexcristici alexcristici self-assigned this May 30, 2024
@alexcristici alexcristici linked an issue May 30, 2024 that may be closed by this pull request
Copy link
Collaborator

@sjg-wdw sjg-wdw left a comment

Choose a reason for hiding this comment

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

Can we prove that this still fixes the original bug?

@alexcristici
Copy link
Collaborator Author

alexcristici commented May 30, 2024

I tested on my iPhone 6s with iOS 15 and it fixes the original bug. I tested without the presentsWithTransaction fix and reproduce the original bug, then tested with presentsWithTransaction fix and could not reproduce it anymore.

@sjg-wdw
Copy link
Collaborator

sjg-wdw commented May 30, 2024

Works for me

@alexcristici
Copy link
Collaborator Author

I added some screen recording.

@alexcristici alexcristici merged commit e082e05 into maplibre:main May 30, 2024
19 checks passed
@alexcristici alexcristici deleted the annotation-view-recycling-re-enabled branch May 30, 2024 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Annotation View recycling on Metal
5 participants