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

Set default cell_size on new TileMap Layer navigation layer maps #79485

Merged
merged 1 commit into from
Jul 24, 2023

Conversation

smix8
Copy link
Contributor

@smix8 smix8 commented Jul 14, 2023

Sets default cell_size on new TileMap Layer navigation layer maps

Related issue #79474.

This was also an issue in 4.0 but that version had no error that made users aware of it.

@smix8 smix8 added bug topic:navigation topic:2d cherrypick:4.0 cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release labels Jul 14, 2023
@smix8 smix8 added this to the 4.2 milestone Jul 14, 2023
@smix8 smix8 requested a review from a team as a code owner July 14, 2023 20:19
scene/2d/tile_map.cpp Outdated Show resolved Hide resolved
@YuriSizov YuriSizov requested review from KoBeWi and groud July 17, 2023 19:20
@YuriSizov YuriSizov changed the title Set default cell_size on new TileMap Layer navigation layer maps Set default cell_size on new TileMap Layer navigation layer maps Jul 21, 2023
@YuriSizov
Copy link
Contributor

Hey, we're ready to merge, but this needs a rebase following #78328 🙏

Sets default cell_size on new TileMap Layer navigation layer maps.
@smix8 smix8 force-pushed the tilemap_cell_size_4.x branch from eb13d31 to 671839d Compare July 21, 2023 16:02
@YuriSizov YuriSizov merged commit 6fb5d9e into godotengine:master Jul 24, 2023
@YuriSizov
Copy link
Contributor

Thanks!

@smix8 smix8 deleted the tilemap_cell_size_4.x branch July 24, 2023 17:48
@YuriSizov YuriSizov removed cherrypick:4.0 cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release labels Sep 21, 2023
@YuriSizov
Copy link
Contributor

So because of #78328 this cannot be easily cherry-picked to 4.1. A dedicated PR would be required if this is desired. (And I wouldn't bother with 4.0, we are unlikely to publish another 4.0.x release at this point).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants