On Mon, 14 Sep 2026 13:45:57 GMT, Lukasz Kostyra <[email protected]> wrote:

>> Andy Goryachev has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   contains
>
> modules/jfx.incubator.richtext/src/test/java/test/jfx/incubator/scene/control/richtext/RichTextAreaNavigationTest.java
>  line 227:
> 
>> 225:         int count = 0;
>> 226:         // this loop should exit as soon as it hits the beginning or 
>> end of the document,
>> 227:         // but let's set a limit, just in case
> 
> Would we want to assert we hit the limit without reaching the end?

it is probably unnecessary since this method checks that the caret moved at 
each step, but it wouldn't hurt either.  added.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2280#discussion_r4007323453

Reply via email to