RFR: 8335587: TextInputControl: Binding prompt text that contains linebreak causes exception [v2]

Andy Goryachev angorya at openjdk.org
Tue Feb 4 18:00:40 UTC 2025


On Tue, 4 Feb 2025 16:01:03 GMT, Ziad El Midaoui <duke at openjdk.org> wrote:

>> When binding the promptTextProperty of a TextInputControl (TextField or TextArea) to a text that contains linebreaks/newlines ("\n") the "bind" call causes a RuntimeException to be thrown, the solution to it is to unbind before calling the set(txt) method to set the new value for the property.
>> Also added tests to test this new fix
>
> Ziad El Midaoui has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix formatting in TextInputControl.java

Looks good in the MonkeyTester.

Thank you for providing unit tests and especially covering a complete set of scenarios (from the applied logic standpoint): null, with, and without newlines.

The fix is localized and well defined, so one reviewer is probably enough, but let's wait the prescribed time period (24h) to give other people a chance to take a look, especially @mstr2 :-)

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

Marked as reviewed by angorya (Reviewer).

PR Review: https://git.openjdk.org/jfx/pull/1694#pullrequestreview-2593641273


More information about the openjfx-dev mailing list