On Thu, 21 Mar 2024 11:56:46 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
>>> I wonder if it's the default focus owner? >> >> It is(e.g. with `setFocusPainted(true)`): >>  >> >>> Should we request focus to Pass instead? >> >> Probably not, to avoid accidentally passing the test, while hitting the >> pause is harmless. > > Not so sure… Pause is not the main control. I guess the focus was in the > instructions previously, we could preserve it. It may be not worth it though. It's not clear why we need a focus on the field with instructions. The focus on the pause button makes sense: Suppose someone is running many, many tests and decides to take a tea break, he just presses the space bar without moving the mouse. (if he knows that the focus is on the pause button, `setFocusPainted(false)` :-) ) Anyway, if this bothers anyone we can come back to it later. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/18368#discussion_r1533887869
