On Mon, 9 Dec 2024 20:09:01 GMT, Harshitha Onkar <[email protected]> wrote:

> Apart from the following suggestions, test changes LGTM and work as expected 
> on x11 & wayland.
> 
> Ln#174: Is it better to move the button click to the center of the btn for 
> stability?
> 
> ```
>  robot.mouseMove(buttonLoc.x + button.getWidth() / 2,
>                  buttonLoc.y + button.getHeight() / 2);
> ```
> 
> Ln#230: `frame = null` is not required in disposeFrame() and can be removed.

ok..good point...updated...

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

PR Comment: https://git.openjdk.org/jdk/pull/22631#issuecomment-2530618905

Reply via email to