On Tue, 13 Oct 2026 13:02:00 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: Split test case for windows / non-windows OS I think it's _good enough_ to go in. This change impacts other functions (word selection, word deletion, etc.) but in a manner consistent with the intent of this PR. What we really need to do in a followup (or followups) is to develop a comprehensive behavior spec, and a set of tests which exercise every single function, on each supported platform, see https://bugs.openjdk.org/browse/JDK-8326869 ------------- Marked as reviewed by angorya (Reviewer). PR Review: https://git.openjdk.org/jfx/pull/2334#pullrequestreview-5507943755
