fantasycalendar / FoundryVTT-Sequencer

This module implements a basic pipeline that can be used for managing the flow of a set of functions, effects, sounds, and macros.
Other
47 stars 24 forks source link

[BUG] - Canvas Freeze on Measurement Controls #250

Closed gambit07 closed 3 months ago

gambit07 commented 3 months ago

Describe the bug Canvas freezes when switching to Measurement Controls for a template placed with a sequencer effect

https://github.com/fantasycalendar/FoundryVTT-Sequencer/assets/4236874/c0825661-4714-4b94-8455-18a14b71966e

Error: sequencer-animation-engine.js:175 Uncaught TypeError: undefined. Cannot create property 'value' on number '1.0175' [Detected 1 package: sequencer] at Object._animateAttribute (sequencer-animation-engine.js:175:41) at sequencer-animation-engine.js:135:9 at Array.forEach (<anonymous>) at Object._animate (sequencer-animation-engine.js:134:6) at sequencer-animation-engine.js:102:48 at Array.forEach (<anonymous>) at Object.nextFrame (sequencer-animation-engine.js:102:20) at mn.emit (TickerListener.ts:71:47) at Pt.update (Ticker.ts:427:37) at Pt._tick (Ticker.ts:129:22)

Sequencer effect: new Sequence() .effect() .file("jb2a.black_tentacles") //assets/animations/animated_spells/black_tentacles_border.webm .scaleToObject(1) .attachTo(template, { align: "right", edge: "on" }) .delay(500) .scaleIn(0, 500, {ease: "easeOutCubic"}) .fadeIn(500) .fadeOut(500) .mask() .belowTokens() .persist() .play()

To Reproduce Steps to reproduce the behavior:

  1. Create a template with the sequencer effect above attached on template creation
  2. Switch to Measurement Controls
  3. Canvas freezes and the error is generated

Setup:

Active modules:

Haxxer commented 3 months ago

Fixed in 3.2.9

gambit07 commented 3 months ago

Fixed in 3.2.9

Thanks! Unfortunately looks like foundry does not like that package update and it won't install

Haxxer commented 3 months ago

Sorry about that, please redownload sequencer, the fixed release should be okay

gambit07 commented 3 months ago

Sorry about that, please redownload sequencer, the fixed release should be okay

Not a problem, appreciate it!