flame icon indicating copy to clipboard operation
flame copied to clipboard

Published 20 hours ago • verified publisher icon flame-engine

feat: Animated tile support!

Open jtmcdole opened this issue 2 years ago • 1 comments

Description

Tiled supports animated tiles, and now Flame does as well...

more_animation

Checklist

  • [x] The title of my PR starts with a [Conventional Commit] prefix (fix:, feat:, docs: etc).
  • [x] I have followed the [Contributor Guide] when preparing my PR.
  • [x] I have updated/added tests for ALL new/updated/fixed functionality.
  • [x] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • [x] I have updated/added relevant examples in examples or docs.

Breaking Change?

  • [ ] Yes, this PR is a breaking change.
  • [x] No, this PR is not a breaking change.

Related Issues

Closes #1914

jtmcdole avatar Sep 21 '22 06:09 jtmcdole

converting to draft: more tests!

  • Testing other map code paths
  • I need to also fix #1932 in this commit.

jtmcdole avatar Sep 21 '22 14:09 jtmcdole

Possible further optimization from discord chat:

  1. add frameTime and frame to TileFrames
  2. update(dt): 1: walk over all TileFrames (very small group), calculatate new frame.
  3. Walk over all TileAnimation (larger group) and update batchedSource if frame value changed

jtmcdole avatar Sep 22 '22 05:09 jtmcdole

Would be good to have an example for the examples directory too.

Need to remember to update it to the newest version of Flame_tiled when it's available.

jtmcdole avatar Sep 23 '22 13:09 jtmcdole

Need to remember to update it to the newest version of Flame_tiled when it's available.

The examples are managed by Melos, so Melos should be updating it automatically once we release a new version.

spydon avatar Sep 23 '22 14:09 spydon

Need to remember to update it to the newest version of Flame_tiled when it's available.

The examples are managed by Melos, so Melos should be updating it automatically once we release a new version.

done!

jtmcdole avatar Sep 23 '22 14:09 jtmcdole