AsepriteNativeParser parsing Forward AnimationTypes to AnimationStrategy.End

[dericatienza]

The current behaviour of AsepriteNativeParser is to set animations' strategy to AnimationStrategy.End when its type is AnimationTypes.Forward. This is inconsistent with. AsepriteJsonParser which sets the strategy to AnimationStrategy.Loop by default or when the tag direction is 'forward'.

I suggest updating AsepriteNativeParser to follow the same behaviour and set animations strategy to AnimationStrategy.Loop by default.

AsepriteNativeParser's strategy setting logic:

excalibur-aseprite/src/AsepriteNativeParser.ts

Lines 96 to 101 in e27ae18

	let strategy = AnimationStrategy.End;
	if (type === AnimationTypes.Forward) {
	strategy = AnimationStrategy.End;
	} else if (type === AnimationTypes.PingPong || type === AnimationTypes.PingPongReverse) {
	strategy = AnimationStrategy.PingPong;
	}

AsepriteJsonParser's strategy setting logic:

excalibur-aseprite/src/AsepriteJsonParser.ts

Lines 54 to 64 in e27ae18

	let strategy = AnimationStrategy.Loop;
	switch(frameTag.direction) {
	case "pingpong": {
	strategy = AnimationStrategy.PingPong
	break;
	}
	case "reverse": {
	frames.reverse();
	break;
	}
	}

[eonarheim]

@dericatienza Good find! Definitely agree and want these to be consistent!

[dericatienza]

Great! I submitted a PR #201.