RFR: 8332895: Support interpolation for backgrounds and borders [v3]
John Hendrikx
jhendrikx at openjdk.org
Fri Aug 2 22:36:57 UTC 2024
On Fri, 2 Aug 2024 21:30:49 GMT, John Hendrikx <jhendrikx at openjdk.org> wrote:
>> Michael Strauß has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - fix since tag
>> - adjust table styling
>
> modules/javafx.graphics/src/main/java/javafx/css/ComponentTransitionable.java line 35:
>
>> 33: * Identifies a class that supports component-wise CSS transitions.
>> 34: * <p>
>> 35: * Component-wise transitions offer more flexibility than {@link Interpolatable} transitions.
>
> Why is the name not `ComponentInterpolatable`?
I see two uses of this interface, both in combination with `Interpolatable`. Should this just extend `Interpolatable`, or are there cases to be expected where this marker is present but `Interpolatable` is not?
Also, have you considered adding a default method to `Interpolatable` to indicate it supports component wise interpolation?
It looks modelled after `RandomAccess`, but the reasons for that being a marker interface are not similar to this situation (it is a marker interface so it can also mark the old collections, like `Vector`).
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1522#discussion_r1702341296
More information about the openjfx-dev
mailing list