<html><head><meta http-equiv="content-type" content="text/html; charset=us-ascii"></head><body style="overflow-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;"><div dir="auto" style="overflow-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;"><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); margin-bottom: 0px; margin-left: 0px; margin-top: 0px;">Hello JavaFX community,</div><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); margin-bottom: 0px; margin-left: 0px; margin-top: 0px;">I've created a draft PR for an enhancement to Transitions: <a aria-label="Link https://github.com/openjdk/jfx/pull/1061" title="https://github.com/openjdk/jfx/pull/1061" href="https://github.com/openjdk/jfx/pull/1061" rel="noreferrer noopener" target="_blank" class="gmail-fui-Link gmail-___1qmgydl gmail-f3rmtva gmail-f1ewtqcl gmail-fyind8e gmail-f1k6fduh gmail-f1w7gpdv gmail-fk6fouc gmail-fjoy568 gmail-figsok6 gmail-f1hu3pq6 gmail-f11qmguv gmail-f19f4twv gmail-f1tyq0we gmail-f1g0x7ka gmail-fhxju0i gmail-f1qch9an gmail-f1cnd47f gmail-fqv5qza gmail-f1vmzxwi gmail-f1o700av gmail-f13mvf36 gmail-f1cmlufx gmail-f9n3di6 gmail-f1ids18y gmail-f1tx3yz7 gmail-f1deo86v gmail-f1eh06m1 gmail-f1iescvh gmail-ftqa4ok gmail-f2hkw1w gmail-fhgqx19 gmail-f1olyrje gmail-f1p93eir gmail-f1h8hb77 gmail-f1x7u7e9 gmail-f10aw75t gmail-fsle3fq" style="overflow-wrap: break-word;">https://github.com/openjdk/jfx/pull/1061</a></div><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); margin-bottom: 0px; margin-left: 0px; margin-top: 0px;">JavaFX contains many different Transitions that have a durationProperty() and related getter/setter.</div><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); margin-bottom: 0px; margin-left: 0px; margin-top: 0px;">Those transitions benefit from an abstract super class which defines this once. </div><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); margin-bottom: 0px; margin-left: 0px; margin-top: 0px;">This will therefore reduce a lot of code duplications or big switch-cases to handle all Transitions like the one linked in the ticket.</div><p style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); margin-bottom: 0px; margin-left: 0px; margin-top: 0px;"> </p><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); margin-bottom: 0px; margin-left: 0px; margin-top: 0px;">I propose to create a new abstract class called TimedTransition, which extends from Transition and defines the durationProperty() + getter/setter. All Transitions which define a durationProperty() will now extend from it and do not need to define it.</div><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); margin-bottom: 0px; margin-left: 0px; margin-top: 0px;">Those Transitions are affected: FadeTransition, FillTransition, PathTransition, PauseTransition, RotateTransition, ScaleTransition, StrokeTransition, TranslateTransition.</div><p style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); margin-bottom: 0px; margin-left: 0px; margin-top: 0px;"> </p><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); margin-bottom: 0px; margin-left: 0px; margin-top: 0px;">Feedback welcome!</div><p style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); margin-bottom: 0px; margin-left: 0px; margin-top: 0px;"> </p><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); margin-bottom: 0px; margin-left: 0px; margin-top: 0px;">Carl</div></div></body></html>