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

AnimatedSprite - SpriteFrames #344

Closed
meniku opened this issue Apr 28, 2014 · 2 comments
Closed

AnimatedSprite - SpriteFrames #344

meniku opened this issue Apr 28, 2014 · 2 comments

Comments

@meniku
Copy link

meniku commented Apr 28, 2014

It would be handy if it was possible to create frames from SpriteSheets / TileSets

@reduz
Copy link
Member

reduz commented Sep 21, 2014

use regular Sprite, which supports spritesheets.

@reduz reduz closed this as completed Sep 21, 2014
@robertdhernandez
Copy link
Contributor

Bumping this issue because I think AnimatedSprite should also support spritesheets.

It's inconsistent for Sprite to handle spritesheets and AnimatedSprite to handle multiple image sprites. Sprite should be just a simple Node that renders a 2D texture with a definable sub region. It shouldn't also do sprite animations -- that's what AnimatedSprite is for: a sprite that has multiple frames defined by a SpriteFrame and animates it very simply. If users want more customized sprite animations, they can use an AnimationPlayer; otherwise the AnimatedSprite handles majority of animated sprite cases in an easy format.

The other issue with Sprite doing animations, is that animating them requires an AnimationPlayer, which becomes really tedious to adjust the Sprite:frame for each key for each animation (unless I'm unaware of a better method).

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

No branches or pull requests

3 participants