> [8236858](https://bugs.openjdk.java.net/browse/JDK-8236858) (Animations do 
> not play backwards after being paused) has been split to deal with 
> [embedded](https://bugs.openjdk.java.net/browse/JDK-8237974) and [not 
> embedded](https://bugs.openjdk.java.net/browse/JDK-8237975) animations. This 
> is a fix for the latter.
> The reason for the split is that embedded animations have a much more complex 
> behavior. The current state of the relation between an animation and its clip 
> envelope is already messy and should be corrected, even more so for embedded 
> animations whose parent controls their behavior as well (sometimes in 
> conflict with the child's clip envelope). This will require a redesign which 
> can be discussed for 15. See the parent issue 
> [8210238](https://bugs.openjdk.java.net/browse/JDK-8210238) for the list of 
> bugs that arise from it.
> 
> This simple fix allows to change the current rate of a `ClipEnvelope` also 
> when the animations is `PAUSED`. A possible issue with this approach is that 
> it changes the buggy behavior of embedded animations to a different buggy 
> behavior.
> 
> A concept test has been added, but it does not work yet since the mock clip 
> envelope does not have sufficient behavior (`doTimePulse` does not actually 
> do a time pulse). Open for ideas on how to make it simple, otherwise I will 
> add a method to set a clip envelope and create a new one ad-hoc.

The pull request has been updated with 1 additional commit.

-------------

Added commits:
 - 317261e6: Implemented test

Changes:
  - all: https://git.openjdk.java.net/jfx/pull/98/files
  - new: https://git.openjdk.java.net/jfx/pull/98/files/54ee486d..317261e6

Webrevs:
 - full: https://webrevs.openjdk.java.net/jfx/98/webrev.01
 - incr: https://webrevs.openjdk.java.net/jfx/98/webrev.00-01

  Stats: 19 lines in 3 files changed: 14 ins; 0 del; 5 mod
  Patch: https://git.openjdk.java.net/jfx/pull/98.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/98/head:pull/98

PR: https://git.openjdk.java.net/jfx/pull/98

Reply via email to