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

Allow opting out of SpriteSheetBundle grid construction #161

Merged
merged 5 commits into from
Feb 9, 2023

Conversation

geieredgar
Copy link
Contributor

@geieredgar geieredgar commented Jan 27, 2023

This is an attempt to fix #105 and fix #151.

This adds an optional no_grid argument to the #[sprite_sheet_bundle] attribute. Instead of using TextureAtlas::from_grid, a TextureAtlas with a single sprite will be used.

Instead of using #[sprite_sheet_bundle(no_grid)], a field of type NoGridSpriteSheetBundle annotated with the #[ldtk_entity] attribute can be used.

Copy link
Owner

@Trouv Trouv left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution, I see now that it would be difficult to support these offset rectangles of tiles with grids.

src/components.rs Outdated Show resolved Hide resolved
src/app/ldtk_entity.rs Outdated Show resolved Hide resolved
src/app/ldtk_entity.rs Outdated Show resolved Hide resolved
examples/basic.rs Outdated Show resolved Hide resolved
Copy link
Owner

@Trouv Trouv left a comment

Choose a reason for hiding this comment

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

LGTM, thanks again

@Trouv Trouv merged commit d6d3c9c into Trouv:main Feb 9, 2023
Trouv added a commit that referenced this pull request Mar 31, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.6.0](v0.5.0...v0.6.0)
(2023-03-31)


### ⚠ BREAKING CHANGES

* In addition to updating to bevy 0.10, users may need define order
between `LdtkSystemSet::ProcessApi` and other 3rd party system sets,
like
[rapier](https://github.com/Trouv/bevy_ecs_ldtk/blob/5b8f17cc51f91ff9aedbed8afca560e750b557c8/examples/platformer/main.rs#L17).
* change LdtkEntity's #[with] attribute to borrow EntityInstance
([#158](#158))
* split `RegisterLdtkObjects` into two new traits with a different
naming convention
([#155](#155))
* change #[from_entity_instance] to use references
([#149](#149))

### Features

* add `#[sprite_sheet_bundle(no_grid)]` attribute for generating a
single-texture `TextureAtlas` instead of a grid
([#161](#161))
([d6d3c9c](d6d3c9c))
* add `with` attribute for LdtkIntCell derive macro
([#157](#157))
([d3fbd3c](d3fbd3c))
* add LevelSet::from_iid method
([#144](#144))
([fb17ae1](fb17ae1))
* add render feature for headless mode (tilemaps only)
([#159](#159))
([2f8000e](2f8000e))
* change #[from_entity_instance] to use references
([#149](#149))
([246880f](246880f))
* change LdtkEntity's #[with] attribute to borrow EntityInstance
([#158](#158))
([c052b31](c052b31))
* register TileMetadata and TileEnumTags types
([#153](#153))
([26cae15](26cae15))
* register types `GridCoords` and `LayerMetadata`
([#146](#146))
([ed4a0f9](ed4a0f9))
* upgrade to bevy 0.10
([#168](#168))
([5b8f17c](5b8f17c))


### Bug Fixes

* use normal sprite for background color instead of tile
([#171](#171))
([b22b11d](b22b11d))


### Example Changes

* improve ground detection in platformer example
([#137](#137))
([cafba57](cafba57))
* use rect_builder buffer instead of row-specific current_rects in
spawn_wall_collisions
([#147](#147))
([45303f3](45303f3))


### Code Refactors

* split `RegisterLdtkObjects` into two new traits with a different
naming convention
([#155](#155))
([156ae8c](156ae8c))


### Documentation Changes

* explain feature flags in crate-level documentation
([#164](#164))
([a832da0](a832da0))
* explain that sprite_bundle should not be used with tilemap editor
visuals ([#142](#142))
([1a7a8a1](1a7a8a1))
* repair doc links to bevy in app module
([#154](#154))
([0f928e8](0f928e8))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Trevor Lovell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants