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

fix: Compute scale in TileLayers based on native map tile size rather than image sizes to support oversized/undersized tiles. #2634

Merged
merged 5 commits into from
Aug 7, 2023

Conversation

chippydip
Copy link
Contributor

@chippydip chippydip commented Aug 1, 2023

Description

Compute isometric tile scale based on the map's tile size rather than the tile image's default size. This fixes an issue where scale was incorrect when those two sizes differed.

Checklist

  • I have followed the Contributor Guide when preparing my PR.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • [-] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • [-] I have updated/added relevant examples in examples or docs.

Breaking Change?

  • Yes, this PR is a breaking change.
  • No, this PR is not a breaking change.

Related Issues

Closes #2633

Compute isometric tile scale based on the map's tile size rather than the tile image's default size. This fixes an issue where scale was incorrect when those two sizes differed.
@chippydip chippydip changed the title fix: Computer scale in IsometricTileLayer based on native map tile size rather than image sizes. fix: Compute scale in IsometricTileLayer based on native map tile size rather than image sizes. Aug 1, 2023
Copy link
Member

@spydon spydon left a comment

Choose a reason for hiding this comment

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

Good find, thanks for the PR!

@chippydip chippydip changed the title fix: Compute scale in IsometricTileLayer based on native map tile size rather than image sizes. fix: Compute scale in TileLayers based on native map tile size rather than image sizes to support oversized/undersized tiles. Aug 4, 2023
@jtmcdole
Copy link
Contributor

jtmcdole commented Aug 6, 2023

LGTM

@spydon spydon enabled auto-merge (squash) August 7, 2023 12:17
@spydon spydon merged commit 1c4d6cd into flame-engine:main Aug 7, 2023
6 checks passed
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.

flame_tiled: Isometric tiles not scaled properly when image size does not match the map tile size.
3 participants