On Wed, 17 Jun 2026 22:00:30 GMT, Andy Goryachev <[email protected]> wrote:

>> Michael Strauß has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   change "since" javadoc tag to 27
>
> modules/javafx.graphics/src/main/java/javafx/scene/layout/HeaderBar.java line 
> 370:
> 
>> 368:      */
>> 369:     public static double getSystemButtonHeight(Stage stage) {
>> 370:         return AttachedProperties.of(stage).systemButtonHeight.get();
> 
> side node: if we decide to put rarely used properties in a hashtable (see 
> https://github.com/andy-goryachev-oracle/Test/blob/main/doc/Experiments/NodeProperties.md
>  ) we could get rid of the `AttachedProperties` machinery (or roll it into 
> the `NodeProperties`).

Yes, attached properties shouldn't be stored in `Node.getProperties()` at all, 
where they are publicly accessible and can also be changed or removed by 
completely unrelated code.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2188#discussion_r3431806524

Reply via email to