On Fri, 25 Oct 2024 14:27:25 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> Andy Goryachev 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 14 additional 
>> commits since the last revision:
>> 
>>  - Merge remote-tracking branch 'origin/master' into ag.text.layout.api
>>  - Merge remote-tracking branch 'origin/master' into ag.text.layout.api
>>  - remove line spacing
>>  - tests
>>  - whitespace
>>  - caret info
>>  - text line info
>>  - Merge remote-tracking branch 'origin/master' into ag.text.layout.api
>>  - convert to wrapper
>>  - clarify
>>  - ... and 4 more: https://git.openjdk.org/jfx/compare/6b8ee716...eb990081
>
> modules/javafx.graphics/src/main/java/javafx/scene/text/LayoutInfo.java line 
> 54:
> 
>> 52:      * <li>{@code minY} is the ascent of the first line (negative)
>> 53:      * <li>{@code width} the width of the widest line
>> 54:      * <li>{@code height} the sum of all lines height
> 
> Is this correct ? What about spacing between lines, or is that included in 
> the line height of each line?

There is an existing related issues
[JDK-8317120](https://bugs.openjdk.org/browse/JDK-8317120) RFE: 
TextFlow.rangeShape() ignores lineSpacing
[JDK-8317122](https://bugs.openjdk.org/browse/JDK-8317122) RFE: 
TextFlow.preferredHeight ignores lineSpacing

so perhaps we need to add a `boolean` argument to qualify whether the return 
value should include the last line's `lineSpacing` or not.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1596#discussion_r1817244811

Reply via email to