On Tue, 14 Apr 2026 17:40:12 GMT, Marius Hanl <[email protected]> wrote:

>> 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`

I just settled on keeping it in line with the rest of the test file. The naming 
for tests is all over the place anyways, so I don't think this matters

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

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

Reply via email to