FlixelCommunity / flixel

Community fork of Adam “Atomic” Saltsman's popular game engine Flixel. Distilled from a variety of Flash games he worked on over the last couple years, including Gravity Hook, Fathom and Canabalt, its primary function is to provide some useful base classes that you can extend to make your own game objects.
http://flixelcommunity.org/
Other
84 stars 17 forks source link

Advanced FlxSprite #201

Closed mezkhalin closed 1 year ago

mezkhalin commented 10 years ago

Adds a subclass of FlxSprite that enables it to support both baked rotation and animations.

Dovyski commented 10 years ago

Nice addition! I'm curious why you decided to create another class instead of merging that feature with FlxSprite itself. Anything in particular?

mezkhalin commented 10 years ago

Well it started out for clarity's sake, but now that you mention it. I'll go ahead and merge it with FlxSprite in the upcoming days.

Gama11 commented 10 years ago

@Dovyski Are you sure it's a good idea to merge this into the core FlxSprite? There are millions of features that could be added to that, and evenutally it would just be a cluttered mess. Not saying this particular feature is useless.. just maybe not that essential, so it has to be considered very well if this should go straight into FlxSprite.

mezkhalin commented 10 years ago

@Gama11 In my point of view this class merges two core functions of FlxSprite; animation and rotation which is why I endorse merging them. I agree that nonessential functions could be collected into an extended class as not to clutter the core class, but I don't consider animation and rotation to be nonessential.