On Fri, 9 Oct 2026 08:59:37 GMT, Robert Lichtenberger <[email protected]> 
wrote:

>> As analysed in the bug, the previousWord() method does not work correctly if 
>> at the word boundary a character is found that is neither letter nor digit.
>> This fix changes the method so that only ranges of whitespace will be 
>> skipped, resulting in correct word selection behaviour.
>> A test case has been added to TextAreaTest.
>> gradlew :controls:test --tests test.javafx.scene.control.Text*Test has been 
>> executed to ensure no regressions happen in other text related classes. No 
>> errors were reported.
>> 
>> 
>> ---------
>> - [X] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Robert Lichtenberger has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   8264588: Disable test that fails under Windows.

Github actions still show failing tests on all the platforms:

<img width="831" height="549" alt="Screenshot 2026-10-12 at 10 48 11" 
src="https://github.com/user-attachments/assets/a1fea787-03d4-4397-986b-ad9201ecbb60";
 />

modules/javafx.controls/src/test/java/test/javafx/scene/control/behavior/TextAreaBehaviorTest.java
 line 128:

> 126:     @Test
> 127:     public void testDoubleClickWithSpecialChars() {
> 128: // disabled, fails under Windows due to OS-specific behaviour in 
> mouseDoubleClick

We don't want to disable this case.  What we need is to create two variants, 
one for Windows and one non-Windows.

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

PR Comment: https://git.openjdk.org/jfx/pull/2334#issuecomment-6129534975
PR Review Comment: https://git.openjdk.org/jfx/pull/2334#discussion_r4250847042

Reply via email to