RFR: 8377153: JavaFX FlowPane layout causing improper wrapping of TextFlow nodes with max-width styling [v3]

John Hendrikx jhendrikx at openjdk.org
Fri Feb 27 17:28:53 UTC 2026


On Fri, 27 Feb 2026 15:29:10 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> John Hendrikx has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove dead code.
>
> modules/javafx.graphics/src/test/java/test/javafx/scene/layout/AsciiArtParser.java line 42:
> 
>> 40:    * @param height the height of the region
>> 41:    */
>> 42:   public record Bounds(double x, double y, double width, double height) {
> 
> should the new code be formatted using 4 spaces per indent?

Ah, oops, I copied this from my personal tools, I'll reformat it.

> modules/javafx.graphics/src/test/java/test/javafx/scene/layout/AsciiArtParser.java line 523:
> 
>> 521:     return extent;
>> 522:   }
>> 523: }
> 
> missing newline

is that a problem?

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2077#discussion_r2865401660
PR Review Comment: https://git.openjdk.org/jfx/pull/2077#discussion_r2865402701


More information about the openjfx-dev mailing list