René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"`TransitionTimeMs` is not inherited from one callback invocation to the next","articleBody":"Inside of a Sprite callback, the API user can set `TransitionTimeMs` to specify how long the transition should take to complete. However, unlike other sprite attributes, `TransitionTimeMs` is not inherited from the previous phase and must be set in each phase callback.\r\n\r\nExample using `Sprite` API  (see comments):\r\n\r\n```ts\r\nconst scene = new Scene({\r\n  defaultTransitionTimeMs: 400,  // Default transition time is 400.\r\n});\r\n\r\nconst sprite = scene.createSprite();\r\n\r\nsprite\r\n  .enter((s: SpriteView) =\u003e {\r\n    s.TransitionTimeMs = 1000;  // Explicitly set time to 1000 (1 sec).\r\n    s.SizeWorld = 1;\r\n    s.FillOpacity = 0;\r\n  })\r\n  .update((s: SpriteView) =\u003e {\r\n    // CURRENTLY: s.TransitionTimeMs implicitly set to the default (400).\r\n    // DESIRED: s.TransitionTimeMs should persist the 1000 from enter().\r\n\r\n    // s.SizeWorld is correctly inherited from previous state (1).\r\n\r\n    s.FillOpacity = 1;\r\n  });\r\n```\r\n\r\nExample using the `Selection` API  (see comments):\r\n\r\n```ts\r\nconst scene = new Scene({\r\n  defaultTransitionTimeMs: 500,  // Default transition time is 500.\r\n});\r\n\r\nconst selection = scene.createSelection();\r\n\r\nselection\r\n  .onInit((s: SpriteView) =\u003e {\r\n    // CURRENTLY: Ignored for rendering and not persisted.\r\n    // DESIRED: Still ignore for rendering, but persist value.\r\n    s.TransitionTimeMs = 1000;  \r\n\r\n    s.SizeWorld = 1;\r\n    s.FillOpacity = 0;\r\n  })\r\n  .onEnter((s: SpriteView) =\u003e {\r\n    // CURRENTLY: s.TransitionTimeMs implicitly set to the default (500).\r\n    // DESIRED: s.TransitionTimeMs should persist 1000 from onInit().\r\n\r\n    s.TransitionTimeMs = 2000;  // Explicitly set to 2000.\r\n\r\n    // s.SizeWorld is inherited from previous state (1).\r\n    s.FillOpacity = 1;\r\n  })\r\n  .onUpdate((s: SpriteView) =\u003e {\r\n    // CURRENTLY: s.TransitionTimeMs implicitly set to the default (500).\r\n    // DESIRED: s.TransitionTimeMs should persist 2000 from onEnter().\r\n\r\n    // s.SizeWorld is inherited from previous state (1).\r\n    s.FillOpacity = 1;\r\n  });\r\n```\r\n\r\nIdeally, `TransitionTimeMs` should persist between callback invocations. The Scene's `defaultTransitionTimeMs` should only be used until the first time a transition time is needed, and then after that the previous value should be used again. This would make it consistent with other sprite attributes.\r\n\r\nNote: The fact that the transition time is ignored for Selection `onInit()` is intentional. The attributes set for `onInit()` should be applied before the first frame in which the Sprite is rendered. However, the `TransitionTimeMs` set during `onInit()` should be used for callback invocations. Ignoring this value and not persisting it is counterintuitive.","author":{"url":"https://github.com/jimbojw","@type":"Person","name":"jimbojw"},"datePublished":"2022-10-11T12:32:57.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/76/megaplot/issues/76"}