On Tue, 6 Oct 2026 18:52:28 GMT, Andy Goryachev <[email protected]> wrote:
>> Florian Kirmaier has updated the pull request with a new target base due to
>> a merge or a rebase. The pull request now contains three commits:
>>
>> - 8388313: worked in some comments and mixed improvements.
>> - 8388313: readded accidentally removed line.
>> - 8388313: Fix redundant evaluation of css when
>> ContextMenu.show(Node,Side,x.y) is called
>
> modules/javafx.controls/src/test/java/test/javafx/scene/control/ContextMenuTest.java
> line 701:
>
>> 699: }
>> 700:
>> 701: @Test public void testCssProcessedOnlyOnce() {
>
> please keep `@Test` annotation on its own line
This is the pattern in the whole class.
I can change it only here, for the whole class - or leave it as it is.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2215#discussion_r4207936015
