Animation repetitions do not work when specifiying animation name

[LazZiya]

The repetitions count is not respected when specifiying animation name.

// This will keep looping
mv.animationName = "other-animation";
mv.play({ repetitions: 1 });

Without animationName it will work fine for the first animaton only.

Live Demo

https://glitch.com/edit/#!/chatter-pumped-armadillo

Version

• model-viewer: v1.10.1

Browser Affected

• Chrome, version: 97.0.4692.99
• Edge
• Safari

OS

• Windows
• iOS

[futahei]

This may be a temporary workaround, but it will work correctly if you do the following this,

mv.animationName = "Death";
await mv.updateComplete;
mv.play({repetitions: 1});

Information about updateComplete is here

[elalish]

Indeed, I wouldn't even call that a temporary workaround so much as just how Lit elements work (we inherit from their library). Everything is async, so await mv.updateComplete is necessary for consecutive operations for which the order is important, particularly attributes. You'll notice it is quite pervasive in our tests.

[LazZiya]

Thank you @futahei , the workaround worked fine 👍

[designbyadrian]

I experienced the same issue with version 1.9.2. Updating to version 1.10.1 solves the issue if I apply the fix above.

[futahei]

@designbyadrian This feature is available since v1.10.0.
Please refer to the release notes v1.10.0.