mannyzhou5 / evolutionchamber

Automatically exported from code.google.com/p/evolutionchamber
0 stars 0 forks source link

Mac support enhancement #102

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I hope that this is the right place for code contributions.  I added better 
integration with Mac OS X.  It basically does three things:

(1) Makes the application name appear in the Mac menu bar (by default, 
"org.eclipse.jdt.internal.jarinjarloader.JarRsrcLoader" appears...this must be 
the class name of the jar-in-jar loader)
(2) Makes an icon appear in the dock.  I just did a Google search for an image 
of the evo chamber and used that.  I also made a change so that the same icon 
also appears in non-Mac OSs (JFrame.setIconImage()).
(3) Allows the application to respond to the user selecting items from the Mac 
application menu, such as "About", "Preferences", and "Quit".

Unfortunately, I had to move the main method (and the checkForUpdates method) 
into a new class (EcSwingXMain) because the Mac code has to run literally 
before *anything* else GUI-related.  Including this code at the beginning of 
the original EcSwingX.main() method does not work, I think because EcSwingX 
extends a class and that class probably does some static initializations.

The Mac code is executed using reflection, so it will still compile and run on 
non-Mac OSs.

All files in the attached ZIP should go in the "com.fray.evo.ui.swingx" 
package.  Thanks.

Original issue reported on code.google.com by mike.angstadt on 3 Nov 2010 at 12:54

Attachments:

GoogleCodeExporter commented 9 years ago
Whoops...accidentally included two files in the ZIP that are not related to 
this issue.  Attached is the correct ZIP.

Original comment by mike.angstadt on 3 Nov 2010 at 1:00

Attachments:

GoogleCodeExporter commented 9 years ago
Lomilar can you grant the commit bit to mike? It would be better to have a 
developer with access to the platform make the changes as he will be able to 
test them properly.
Also added OpSys-All tag because the icon is set for all platforms.

Original comment by netpr...@gmail.com on 3 Nov 2010 at 11:35

GoogleCodeExporter commented 9 years ago
Please send me an email, fritley@gmail.com, with your gmail account. Or drop it 
to me on IRC.

Original comment by Frit...@gmail.com on 6 Nov 2010 at 1:25

GoogleCodeExporter commented 9 years ago
Hey are you guys still interested in this?  I'd love to commit these changes. :)

Original comment by mike.angstadt on 9 Nov 2010 at 4:45

GoogleCodeExporter commented 9 years ago
mike, they asked you to email them so you can commit the changes yourself.

Original comment by dgcastel...@gmail.com on 9 Nov 2010 at 10:13

GoogleCodeExporter commented 9 years ago

Original comment by mike.angstadt on 9 Nov 2010 at 10:29

GoogleCodeExporter commented 9 years ago
Ok, yup, Lomilar gave me commit access. :)

Original comment by mike.angstadt on 9 Nov 2010 at 10:30