Applet in JFrame, JMenu problem

[Bill86]

I'm having a problem with the below code, being displayed properly on windows vista. However it works on windows xp for me.

public GUI(Applet app) {
	super("Title");
	JPanel panel = new JPanel();
	panel.setLayout(new BorderLayout());
	
	JMenuBar jmb = new JMenuBar();
	
	// -- File --
	JMenu file = new JMenu("File");
		JMenuItem open   = new JMenuItem("Open...");
		
		open.setActionCommand("file-open");
		open.addActionListener(this);
		
		JMenuItem reload = new JMenuItem("Reload");
		
		reload.setActionCommand("file-reload");
		reload.addActionListener(this);
		
		JMenuItem exit         = new JMenuItem("Exit");
		
		exit.setActionCommand("file-exit");
		exit.addActionListener(this);
		
		file.add(open);
		file.add(reload);
		
		file.add(new JSeparator());
		
		file.add(exit);
	// --
	
	jmb.add(file);
	
	getContentPane().add(jmb);
	setJMenuBar(jmb);
	getContentPane().add(app, BorderLayout.CENTER);
	setSize(519, 394);
	setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	setResizable(false);
	setVisible(true);
}

What happens is:

On vista, the JMenuItem's are displayed under the applet. Instead of on top of the applet.

Why is this?

Thanks,
Bill86

[Fubarable]

It seems funny that you're adding an Applet to to a JFrame, but perhaps your problem is more from mixing AWT and Swing components than from trying shove the applet square peg into the desktop round hole.

So why this design?

[Bill86]

It seems funny that you're adding an Applet to to a JFrame, but perhaps your problem is more from mixing AWT and Swing components than from trying shove the applet square peg into the desktop round hole.

So why this design?

The applet is from a different java program. That I'm displaying on the JFrame.

What should I use instead? To prevent "mixing AWT and Swing components".

[Fubarable]

You would be better off if the "different Java program" (here called the "GUI class") produced a JPanel instead of an Applet. This way, should you desire to display it as an applet, you would create a class that extends JApplet, have it hold an instance of your GUI class (the one that now produces a JPanel) and place the JPanel into the JApplet's contentPane. Likewise if you wanted to display it as a JFrame, create a class that displays a JFrame, have it hold an instance of the GUI class and then place the GUI's JPanel into the JFrame's contentPane.

[Bill86]

Like this?


"GUI.class" that now extends JPanel, instead of JFrame.

        // Applet app = (Applet) Class.forName("abc", false, loader).newInstance();
	public GUI(Applet app) {
		setLayout(new BorderLayout());
		add(app, BorderLayout.CENTER);
	}

The below class extends JFrame / implements ActionListener.

        // 'panel' = an instance of the "GUI.class".
	public GUIJ(JPanel panel) {
		super("Title");
		
		JMenuBar jmb = new JMenuBar();
		
		// -- File --
		JMenu file = new JMenu("File");
			JMenuItem open   = new JMenuItem("Open...");
			
			open.setActionCommand("file-open");
			open.addActionListener(this);
			
			JMenuItem reload = new JMenuItem("Reload");
			
			reload.setActionCommand("file-reload");
			reload.addActionListener(this);
			
			JMenuItem exit         = new JMenuItem("Exit");
			
			exit.setActionCommand("file-exit");
			exit.addActionListener(this);
			
                        file.add(open);
                        file.add(reload);

			file.add(new JSeparator());
			
			file.add(exit);
		// -- End --
			
		jmb.add(file);
		getContentPane().add(jmb);
		setJMenuBar(jmb);
		getContentPane().add(panel);
		
		setResizable(false);
		setSize(519, 394);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setVisible(true);
	}

It works just the same on xp, I'll test it on vista when I can.

Edit: Still doesn't work on vista. Am I doing something wrong still?

[Fubarable]

Like this?
"GUI.class" that now extends JPanel, instead of JFrame.
.....

It works just the same on xp, I'll test it on vista when I can.

Edit: Still doesn't work on vista. Am I doing something wrong still?

The biggest problem is the Applet as this is still an AWT component. So if possible this is the class that should produce a JPanel.