On Tue, 14 Apr 2026 17:37:23 GMT, Andy Goryachev <[email protected]> wrote:

>> Christopher Schnick has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Add tests for centering
>
> modules/javafx.controls/src/test/java/test/javafx/scene/control/AlertTest.java
>  line 341:
> 
>> 339:     }
>> 340: 
>> 341:     @Test public void alert_owner_iconifiedCentering() {
> 
> minor: we probably should place `@Test` annotations on its own line in the 
> new code.

and maybe use camelCase completely? Since it is already used inside 
`iconifiedCentering`

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2127#discussion_r3081342093

Reply via email to