On Fri, 2 Aug 2024 02:13:16 GMT, Michael Strauß <mstra...@openjdk.org> wrote:
>> modules/javafx.graphics/src/main/java/javafx/scene/paint/Color.java line >> 1956: >> >>> 1954: >>> 1955: // If both instances are equal, return this instance to >>> prevent the creation of numerous small objects. >>> 1956: if (t <= 0.0 || equals(endValue)) return this; >> >> This question might be outside of the current PR, but I think the naive >> color interpolation algorithm is a bad choice here (L1960). >> >> See for example >> https://www.alanzucconi.com/2016/01/06/colour-interpolation/ >> >> The question that is relevant to this PR is whether we should enable the >> user to specify the interpolation algorithm. > > CSS Transitions 1 is a feature-complete spec, and it doesn't include an > option to specify the transition color space. That might come with CSS > Transitions 2, but it's not there yet. I think we should revisit this once > the feature has landed in the spec. pardon my ignorance, could you clarify what a "CSS Transition 1" is? ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1522#discussion_r1701960667