RFR: 8237975: Non-embedded Animations do not play backwards after being paused
Nir Lisker
nlisker at openjdk.java.net
Tue Jan 28 19:30:51 UTC 2020
[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.
-------------
Commits:
- 54ee486d: Remove unused import
- 355d04f0: Added concept test
- 60a57f9a: Finalize fix
- d8de2ea5: cont.
- 4d34544d: Initial commit
Changes: https://git.openjdk.java.net/jfx/pull/98/files
Webrev: https://webrevs.openjdk.java.net/jfx/98/webrev.00
Issue: https://bugs.openjdk.java.net/browse/JDK-8237975
Stats: 26 lines in 5 files changed: 17 ins; 6 del; 3 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
More information about the openjfx-dev
mailing list