On Thu, 6 Jun 2024 18:18:14 GMT, Damon Nguyen <dngu...@openjdk.org> wrote:
>> Alisen Chung has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> add event check in popup trigger
>
> src/java.desktop/macosx/classes/sun/lwawt/macosx/NSEvent.java line 276:
>
>> 274: && jeventType != MouseEvent.MOUSE_RELEASED) {
>> 275: return false;
>> 276: }
>
> Just to double check, are the only jeventTypes to trigger popups using mouse?
> Could there be other event types, maybe using keyboard for example with
> tab/arrows & enter?
I believe only MouseEvents can be popup triggers. I checked in MouseEvent that
there is a field for popup triggers and no such field in KeyEvent.
The isPopupTrigger method here also previously checked button down masks
already, so adding a check for the correct mouse events shouldn't be an issue
even if there was a way to show a popup menu with a different event type.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19569#discussion_r1630039417
