RFR: 8327838: Convert java/awt/FileDialog/MultipleMode/MultipleMode.html applet test to main [v2]

Alexey Ivanov aivanov at openjdk.org
Tue Mar 12 09:51:15 UTC 2024


On Tue, 12 Mar 2024 05:17:31 GMT, Tejesh R <tr at openjdk.org> wrote:

>> Alexander Zvegintsev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review comments
>
> test/jdk/java/awt/FileDialog/MultipleMode.java line 101:
> 
>> 99:         frame.add(sysout, BorderLayout.CENTER);
>> 100: 
>> 101:         frame.pack();
> 
> Frame size is too big and going out of the screen, might fix a size here.

It doesn't fit on the screen of my VM either.

One way to resolve this problem is to add

            .position(PassFailJFrame.Position.TOP_LEFT_CORNER)

to `PassFailJFrame` builder as there's no easy way to move the instruction frame to the left so that both frames fit on the screen.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18205#discussion_r1521162323


More information about the client-libs-dev mailing list