tilesets: only set autoAnimation on tiles that need it

city41 · Dec 10, 2023 · 57501f0 · 57501f0
1 parent 1a66b4f
commit 57501f0
Showing 1 changed file with 11 additions and 4 deletions.
diff --git a/src/generators/tilesets/tilesets.ts b/src/generators/tilesets/tilesets.ts
@@ -27,10 +27,17 @@ function applyChildTiles(
 	masterFrame: CROMTile[][],
 	childFrames: CROMTile[][][]
 ) {
-	for (let f = 0; f < childFrames.length; ++f) {
-		for (let y = 0; y < childFrames[f].length; ++y) {
-			for (let x = 0; x < childFrames[f][y].length; ++x) {
-				if (masterFrame[y][x]) {
+	for (let y = 0; y < masterFrame.length; ++y) {
+		for (let x = 0; x < masterFrame[y].length; ++x) {
+			// if they are all the same, no need to make this an autoAnimation
+			const allFrameTilessAreEqual = childFrames.every((cf) => {
+				return (
+					cf[y][x].canvasSource.toDataURL() ===
+					masterFrame[y][x].canvasSource.toDataURL()
+				);
+			});
+			if (!allFrameTilessAreEqual) {
+				for (let f = 0; f < childFrames.length; ++f) {
 					// @ts-expect-error it wants this array to already be 3 or 7 in size
 					masterFrame[y][x].childAnimationFrames ??= [];
 					masterFrame[y][x].childAnimationFrames!.push(childFrames[f][y][x]);