Skip to content

Conversation

ReinisSprogis
Copy link
Contributor

Previously each tile widget was discarded and rebuilt on every map gesture. Now it just gets repositioned. I still need to check if this doesn't cause any implications. I think it should improve performance, but "I think" is not a metric, so still want to check on that.

@ReinisSprogis
Copy link
Contributor Author

I put the tile into Repaint boundry to check how often it is rebuilt nad was showing on every gesture init was called and dispose.

2025-07-13.14-32-20.mp4

The new approach preservs tiles state.

2025-07-13.14-34-16.mp4

What you think of this?

@JaffaKetchup
Copy link
Member

Hey, I'm away right now but this looks pretty good! I'll take a look at it when I'm back. Nice mid-way point (IYSWIM) as we start work on the new tile layer stuff 👍

@JaffaKetchup
Copy link
Member

Thanks for the PR. Unfortunately in my own testing I could not necessarily see a significant performance improvement. I agree that in theory, I would expect this to be better, but it does not seem to necessarily be the case.
I think I also saw some white hairline fractures between tiles, especially in debug mode. I will have a look more, but it may just be worth waiting for the new tile layer at this point, not sure.

@ReinisSprogis
Copy link
Contributor Author

ReinisSprogis commented Jul 22, 2025

Yeah. I had the same feeling. And with new TileLayer on horizon, might not be worth it.

Might be better for power consumption if not for performance, but have not tested.

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

Successfully merging this pull request may close these issues.

2 participants