RFR: 8313424: JavaFX controls in the title bar [v22]
Michael Strauß
mstrauss at openjdk.org
Sun Mar 23 08:59:23 UTC 2025
On Tue, 5 Nov 2024 23:00:36 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> Michael Strauß has updated the pull request incrementally with three additional commits since the last revision:
>>
>> - HeaderBar changes
>> - EMPTY Dimension2D constant
>> - use CsvSource in HeaderBarTest
>
> modules/javafx.graphics/src/test/java/test/javafx/scene/layout/HeaderBarTest.java line 237:
>
>> 235: "BOTTOM_RIGHT, 740, 40, 100, 50"
>> 236: })
>> 237: void alignmentOfCenterChild_notResizable_withNonEmptyLeadingAndTrailingChild(
>
> minor suggestion:
>
> long test names do not help much (and actually may cause problems, see https://bugs.openjdk.org/browse/JDK-8334497)
>
> also rather inconvenient to see in the logs, or IDEs. You can use comments for human-readability and short(er) method names.
I've thought about it for a bit, and decided to leave it as-is for this class.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1605#discussion_r2009051894
More information about the openjfx-dev
mailing list