fix!: remove `LdtkAsset::world_height` and correct `UseWorldTranslation` y-calculation - Githubissues

Trouv / bevy_ecs_ldtk

ECS-friendly ldtk plugin for bevy, leveraging bevy_ecs_tilemap

Other

641 stars 73 forks source link

fix!: remove `LdtkAsset::world_height` and correct `UseWorldTranslation` y-calculation #207

Closed Trouv closed 1 year ago

Trouv commented 1 year ago

The y-values used for spawning levels in the UseWorldTranslation case were incorrect. There were a couple problems:

the world translation in LDtk was treated as relative to the height of the world, when it was actually relative to the origin
the method for getting the world height LdtkAsset::world_height was incorrect, instead calculating more of a "world depth"

BREAKING-CHANGE: LdtkAsset::world_height has been removed