On Wed, 24 Dec 2025 17:26:40 GMT, Christopher Schnick <[email protected]> wrote:

>> This is a pretty simple fix for the accessible text of password fields.
>
> Christopher Schnick has updated the pull request incrementally with two 
> additional commits since the last revision:
> 
>  - Add test
>  - Fix format

modules/javafx.controls/src/test/java/test/javafx/scene/control/PasswordFieldTest.java
 line 101:

> 99:     }
> 100: 
> 101:     @Test public void accessibleAttributeText() {

Thank you for adding the test!

we usually place `@Test` on a separate line (I know you followed the existing 
style).

also, please update the copyright year to 2025.

otherwise, looks good.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2020#discussion_r2646095996

Reply via email to