I want you to be the one who commits to the repository but before you do that please attach the patch here so that I can review the changes or send it to me privately via e-mail. Thanks a lot Magne!
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
I also updated the way the main window is handled by the onClick listener on the tray icon to use SwingUtilitis.invokeLater() method (not really related to this issue, I can strip it out if you want me to)
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Patch for this bug has been applied to branch cesilko/bug178. However on systems with multiple monitors this improvement for some reason causes serious problems during startup due to JDK-8206266 [1] - main window does not load or system tray icon does not get its popup menu. This needs more investigation before merging to master.
I tried it myself and it really does not work in Rachota 2.4 build #110628. Changing to bug.
Shouldn't be hard to fix, I can see what I can do about it :)
Thanks Magne! BTW, the feature was contributed in RFE #2990706.
The patch from RFE #2990706 doesn't cleanly apply to the current dev. branch...
I will have a look at what he did, and manually duplicate it in the code.
When I am done, do you want me to post the patch here, or do you want me to push it to the repository?
I want you to be the one who commits to the repository but before you do that please attach the patch here so that I can review the changes or send it to me privately via e-mail. Thanks a lot Magne!
2011-07-19T21:35.patch
Ah, forgot to add the comment.
The attached patch solves this issue.
I also updated the way the main window is handled by the onClick listener on the tray icon to use SwingUtilitis.invokeLater() method (not really related to this issue, I can strip it out if you want me to)
Had a chance to look at the patch yet?
Patch for this bug has been applied to branch cesilko/bug178. However on systems with multiple monitors this improvement for some reason causes serious problems during startup due to JDK-8206266 [1] - main window does not load or system tray icon does not get its popup menu. This needs more investigation before merging to master.
[1] https://bugs.openjdk.java.net/browse/JDK-8206266