RFR: 8339984: Open source AWT MenuItem related tests [v7]
Abhishek Kumar
abhiscxk at openjdk.org
Wed Sep 18 12:35:35 UTC 2024
On Wed, 18 Sep 2024 12:26:19 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>>> Actually, I thought `.positionTestWindow` would be called for the first window automatically, then you can position other windows based on the assigned location. Eventually, it's not called at all if a list of windows is created.
>>>
>>> I'll submit a bug for it.
>>
>> Submitted [JDK-8340365](https://bugs.openjdk.org/browse/JDK-8340365): _Position the first window of a window list_.
>
>> > Actually, I thought `.positionTestWindow` would be called for the first window automatically, then you can position other windows based on the assigned location. Eventually, it's not called at all if a list of windows is created.
>> > I'll submit a bug for it.
>>
>> Submitted [JDK-8340365](https://bugs.openjdk.org/browse/JDK-8340365): _Position the first window of a window list_.
>
> If that had been the case, you would've used `componentMoved` and the code would've been shorter by 1 line as shown in #21057:
>
>
> @Override
> public void componentMoved(ComponentEvent e) {
> testFrame.setLocation(firstFrame.getX(),
> firstFrame.getY() + firstFrame.getHeight() + 8);
> }
I saw your PR and to reflect the changes here https://github.com/openjdk/jdk/pull/21057 should get integrated first.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21029#discussion_r1764971651
More information about the client-libs-dev
mailing list