Ticket #1840 (new defect)

Opened 6 years ago

Last modified 6 years ago

Menu dispatching broken on graceful-exit branch

Reported by: bdezonia Owned by: curtis
Priority: major Milestone: imagej2-b10-compatibility
Component: UI Framework Version:
Severity: serious Keywords:
Cc: Blocked By:
Blocking: #1356

Description

The graceful-exit branch was created to fix #1835. Menus are now updated on WinActivatedEvents. However this is somewhat problematic. For instance use mouse to load two or more sample images quickly. Often you'll get an exception and one of the images doesn't load (and you can no longer exit the program). The exception is related to the menu. ctrueden had an idea from stackoverflow on how to have one menu bar rather than one per window. Either we need to implement that or fix how menus are synced to avoid this exception. And we must decide when to merge graceful-exit to master.

Exception:

2013-04-25 13:17:06.855 java[27655:903] java.lang.IllegalArgumentException: null source

at java.util.EventObject.<init>(EventObject.java:38)
at java.awt.AWTEvent.<init>(AWTEvent.java:315)
at java.awt.event.InvocationEvent.<init>(InvocationEvent.java:185)
at java.awt.event.InvocationEvent.<init>(InvocationEvent.java:153)
at sun.awt.PeerEvent.<init>(PeerEvent.java:25)
at sun.awt.PeerEvent.<init>(PeerEvent.java:20)
at sun.awt.SunToolkit.executeOnEventHandlerThread(SunToolkit.java:582)
at apple.awt.CMenuItem.handleAction(CMenuItem.java:115)

Change History

comment:1 Changed 6 years ago by bdezonia

  • Blocking 1563 added

comment:2 Changed 6 years ago by curtis

  • Component changed from Core to UI Framework
  • Blocking 1356 added; 1563 removed
  • Milestone changed from imagej2-b7-ndim-data to imagej2-b10-compatibility

I  took a crack at addressing this issue, using the modeless JDialog approach discussed at  SO #16175884. But doing it right (which we ultimately must do) is too involved for beta7.

Last edited 6 years ago by curtis (previous) (diff)

comment:3 Changed 6 years ago by bdezonia

Okay. Note I forgot to update this ticket to note that graceful-exit was merged some time ago.

Note: See TracTickets for help on using tickets.