RFR: JDK-8245919: Region#padding property rendering error [v3]
Marius Hanl
mhanl at openjdk.org
Mon May 15 17:59:53 UTC 2023
On Mon, 15 May 2023 16:44:58 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> John Hendrikx has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>>
>> - Merge branch 'openjdk:master' into feature/css-cache-bug
>> - Fix typo
>> - Fix bug in CSS caching code that could reset values on unrelated nodes
>>
>> The bug occurs due to a cache entry being constructed incorrectly when
>> the initial node that triggered the cache entry creation has user set
>> values. The calculated values for properties with a user set value were
>> omitted in the cache entry, and other nodes that later share the same
>> entry would incorrectly assume the omitted property was unstyled and
>> were therefore reset to their default values.
>
> modules/javafx.graphics/src/test/java/test/javafx/scene/CssStyleHelperTest.java line 622:
>
>> 620: }
>> 621: """
>> 622: );
>
> is there a chance we would backport this fix (to < java15)?
Even if there is, this still shouln't stop us using new language features, should it? Otherwise we can't really use any new feature, as a fix *might* be backported.
On the other hand it is also easy to replace this text block when backported.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1072#discussion_r1194170939
More information about the openjfx-dev
mailing list