-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Deprecate SpriteSheetBundle
and AtlasImageBundle
#12218
Conversation
AtlasImageBundle
and SpriteSheetBundle
SpriteSheetBundle
and AtlasImageBundle
Had to add |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot! I am in favour of this. I just left a couple of comments about the documentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks a lot for working on this!
# Objective After the `TextureAtlas` changes that landed in 0.13, `SpriteSheetBundle` is equivalent to `TextureAtlas` + `SpriteBundle` and `AtlasImageBundle` is equivalent to `TextureAtlas` + `ImageBundle`. As such, the atlas bundles aren't particularly useful / necessary additions to the API anymore. In addition, atlas bundles are inconsistent with `ImageScaleMode` (also introduced in 0.13) which doesn't have its own version of each image bundle. ## Solution Deprecate `SpriteSheetBundle` and `AtlasImageBundle` in favor of including `TextureAtlas` as a separate component alongside `SpriteBundle` and `ImageBundle`, respectively. --- ## Changelog - Deprecated `SpriteSheetBundle` and `AtlasImageBundle`. ## Migration Guide - `SpriteSheetBundle` has been deprecated. Use `TextureAtlas` alongside a `SpriteBundle` instead. - `AtlasImageBundle` has been deprecated. Use `TextureAtlas` alongside an `ImageBundle` instead.
Objective
After the
TextureAtlas
changes that landed in 0.13,SpriteSheetBundle
is equivalent toTextureAtlas
+SpriteBundle
andAtlasImageBundle
is equivalent toTextureAtlas
+ImageBundle
. As such, the atlas bundles aren't particularly useful / necessary additions to the API anymore.In addition, atlas bundles are inconsistent with
ImageScaleMode
(also introduced in 0.13) which doesn't have its own version of each image bundle.Solution
Deprecate
SpriteSheetBundle
andAtlasImageBundle
in favor of includingTextureAtlas
as a separate component alongsideSpriteBundle
andImageBundle
, respectively.Changelog
SpriteSheetBundle
andAtlasImageBundle
.Migration Guide
SpriteSheetBundle
has been deprecated. UseTextureAtlas
alongside aSpriteBundle
instead.AtlasImageBundle
has been deprecated. UseTextureAtlas
alongside anImageBundle
instead.