On Wed, 8 Apr 2026 09:37:06 GMT, Jose Pereda <[email protected]> wrote:

> Not sure what you mean about the system menu being canceled?

Sorry, I should have been clearer. The call to `[_menu 
cancelTrackingWithoutAnimation]` will cause the menu to not actually open but 
this PR will still send the jMenuOpeningMethod event. Since the menu never 
opened it will never close so there won't be a matching jMenuClosedMethod event.

> Should I go ahead and change it removing the notification in favour of the 
> grab?

Yes, I think you should remove it. There might be some strange case where 
s_grabWindow isn't set to nil when it should be and we'll cancel the opening of 
a system menu unexpectedly. But the chance of that is very small and the 
problem would correct itself in a single click.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2102#discussion_r3052089747

Reply via email to