RFR: 8332895: Support interpolation for backgrounds and borders [v18]

John Hendrikx jhendrikx at openjdk.org
Sat Aug 10 11:36:38 UTC 2024


On Sat, 10 Aug 2024 11:24:34 GMT, John Hendrikx <jhendrikx at openjdk.org> wrote:

>> I also think that we're probably dealing with two different things that are mixed up in `StyleConverter`. But instead of having this discussion here, it probably makes more sense to have it in the context of [JDK-8333121](https://bugs.openjdk.org/browse/JDK-8333121) because it seems relevant to eventually support both short-hand and long-hand property notations.
>> 
>> I've renamed `WithReconstructionSupport` to `SubPropertyConverter` and moved it to an internal package for now. Since both `BackgroundConverter` and `BorderConverter` are not public, we can change it any way we want at a later time.
>
> Yes, that's a good idea, I was mainly worried about introducing a new public API that we can't remove later when we haven't figured out yet what the best direction would be.  With this API removed, I think we can get this merged.

One more thing.  You introduced `transition` short hand version.  Is it maybe a good idea to not do this yet until we have a better idea how to deal with them?  Shorthand properties are just a convenience; the feature will work without them.  If it was already released, then can ignore this as we're too late.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1522#discussion_r1712628315


More information about the openjfx-dev mailing list