Skip to content

fixed setting currentTime#4231

Merged
elalish merged 2 commits intomasterfrom
setTime
Apr 19, 2023
Merged

fixed setting currentTime#4231
elalish merged 2 commits intomasterfrom
setTime

Conversation

@elalish
Copy link
Contributor

@elalish elalish commented Apr 19, 2023

Fixes #4226

Just a (almost) one-liner, along with ditching the useless warning. It's required to set animation-name="0" (or any other name) before setting currentTime will work. Otherwise the model stays at resting pose.

@elalish elalish requested a review from diegoteran April 19, 2023 20:33
@elalish elalish self-assigned this Apr 19, 2023
diegoteran
diegoteran previously approved these changes Apr 19, 2023
@elalish elalish merged commit f22963d into master Apr 19, 2023
@elalish elalish deleted the setTime branch April 19, 2023 22:31
JL-Vidinoti pushed a commit to vidinoti/model-viewer that referenced this pull request Apr 22, 2024
* fixed set time

* revert to initial pose
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants