On Mon, 31 Aug 2026 23:17:43 GMT, Jose Pereda <[email protected]> wrote:
>> Fixes broken navigation introduced in >> [JDK-8370902](https://bugs.openjdk.org/browse/JDK-8370902) with non-zero >> paragraph spacing (above, below, line spacing). >> >> Updated the Rich Editor Demo to allow setting paragraph line spacing in >> addition to space above/below (right click -> Paragraph). >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > modules/jfx.incubator.richtext/src/main/java/com/sun/jfx/incubator/scene/control/richtext/RangeInfo.java > line 69: > >> 67: // remove line spacing from the last line to force navigating to >> the next cell >> 68: if (sz > 0) { >> 69: d[sz - 1] -= lineSpacing; > > if size of `d` is `2 * sz`, this should be > > d[2 * sz - 1] -=... > ``` > or `d[d.length - 1]`. back in the olden days, sz+sz was much faster than sz*2... with your permission, I'll leave it as is ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2280#discussion_r3907491345
