On Thu, 15 Feb 2024 23:25:41 GMT, Alisen Chung <[email protected]> wrote:
>> I updated and ran the test changes. The instructions seem unclear. I double
>> right click the icon in the icon tray and the icon temporarily disappears,
>> then reappears. I think that's the correct behavior according to "After each
>> action event the tray icon is removed from the tray and then added back in a
>> second."
>>
>> Also the instructions describe that there should be some "brief information
>> about action events printed below". But I don't see any info printed.
>>
>> Can you clear this up?
>
>> I updated and ran the test changes. The instructions seem unclear. I double
>> right click the icon in the icon tray and the icon temporarily disappears,
>> then reappears. I think that's the correct behavior according to "After each
>> action event the tray icon is removed from the tray and then added back in a
>> second."
>>
>> Also the instructions describe that there should be some "brief information
>> about action events printed below". But I don't see any info printed.
>>
>> Can you clear this up?
>
> I've fixed the test outputs to go into the frame instead of sysout, so you
> should be able to see the information about the action events in the frame
> that pops up now. I've also updated the instructions to reflect this
> @alisenchung Updated changes look good. I have one clarification though : Why
> is System.setProperty("sun.awt.exception.handler\n",
> "DisposeInActionEventTest$EDTExceptionHandler") used in this test?
The line was in the original test, but it looks like it's not needed so I'll
remove it
-------------
PR Comment: https://git.openjdk.org/jdk/pull/17838#issuecomment-1965209798
