Problem mit PopupMenu und eigenem Panel

[FreeCastle]

Hi,

ich habe mir eine eigene Klasse erstellt, welche von JPanel erbt. Als Zusatz hat diese Klasse ein Attribut welches ein Image enthält, dass auf dem Panel gezeichnet wird. Das sieht dann vereinfacht so aus:

public class ImagePanel extends JPanel {

	private Image content;
	private int width,height;
	
	public ImagePanel(Image image, int w, int h) {
		super();
		content = image;
		width  = w;
		height = h;
	}

	public void paintComponent(Graphics g) {
		super.paintComponent(g);
		g.drawImage(content,0,0,width,height,null);
	}

	public Dimension getPreferredSize() {
		return new Dimension(width,height);
	}

}

So weit so gut. Wie man sieht habe ich die paintComponent-Methode überschrieben, so dass das Image auch tatsächlich gezeichnet wird.
Jetzt wollte ich allerdings noch ein Popup-Menu hinzufügen, das aufgerufen wird sobald man einen Rechtsklick auf so ein Panel macht. Das sieht dann so aus:

public class ImagePanel extends JPanel {

	private Image content;
	private int width,height;

	private JPopupMenu popup;
	private JMenuItem  rotateItem,renameItem;
	
	public ImagePanel(Image image, int w, int h) {
		super();
		content = image;
		width  = w;
		height = h;

		popup = new JPopupMenu();
		rotateItem = new JMenuItem("Rotate...");
		renameItem = new JMenuItem("Rename...");
		popup.add(rotateItem);
		popup.add(renameItem);
	
		this.add(popup);
		this.addMouseListener(new MouseAdapter() {
			public void mouseClicked(MouseEvent me) {
				if ( me.isPopupTrigger() ) {
					popup.show(me.getComponent(),me.getX(),me.getY());
				}
			}

		});
	}

	public void paintComponent(Graphics g) {
		super.paintComponent(g);
		g.drawImage(content,0,0,width,height,null);
	}

	public Dimension getPreferredSize() {
		return new Dimension(width,height);
	}

}

Wie man sieht hat sich nicht viel verändert, ich füge halt das simple Popup-Menü zum Panel hinzu und erstelle dafür einen MouseListener. Das Problem ist jetzt aber, dass das Popup nicht wirklich erscheint wenn ich einen Rechtsklick mache. Das geht nur ganz langsam und "verkrüppelt". Das Problem liegt, so denke ich, daran dass ich eben ein eigenes Panel hab und die paintComponent-Methode überschrieben habe. Ich habe deshalb nämlich mal folgendes in der paintComponent-Methode ausprobiert:

	public void paintComponent(Graphics g) {
		super.paintComponent(g);
		g.drawImage(content,0,0,width,height,null);
		//if (popup.isVisible())
		//if (popup.isShowing())
			popup.paintComponents(g);
	}

So wird das Popup-Menü *immer* sichtbar, und wenn ich dann einen Rechtsklick auf das entsprechende Panel mache dann funktioniert es einwandfrei und flüssig. Nur stört jetzt eben die Tatsache, dass schon immer das PopupMenü sichtbar ist. Ich habs auch schon mit den beiden oben auskommentierten Ifs mal probiert, aber das hat auch nix gebracht. Wie kann ich also ein normales Popupmenü-Verhalten erreichen ? Wär super wenn mir da jemand helfen könnte...

 

[André Uhres]

Versuch's mal mit "mouseReleased" statt "mouseClicked":

/*
 * PopupImagepanelDemo.java
 */
package paint;
import java.awt.*;
import java.awt.event.*;
import java.io.*;
import java.net.*;
import javax.imageio.*;
import javax.swing.*;
public class PopupImagepanelDemo extends JFrame {
    private Image img;
    public PopupImagepanelDemo() {
        super("PopupImagepanelDemo");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(300,200);
        setLocationRelativeTo(null);
//        URL url = getClass().getResource("cows.jpg");
        URL url = null;
        try {
            url = new URL("http://images.thetimes.co.uk/TGD/picture/0,,223788,00.jpg");
        } catch (MalformedURLException ex) {
            ex.printStackTrace();
        }
        try {
            img = ImageIO.read(url);
        } catch (IOException e) {e.printStackTrace();}
        ImagePanel ip = new ImagePanel(img, img.getWidth(null), img.getHeight(null));
        add(ip);
    }
    public static void main(String args[]) {new PopupImagepanelDemo().setVisible(true);}
}
class ImagePanel extends JPanel {
    private Image content;
    private int width,height;
    private JPopupMenu popup;
    private JMenuItem  rotateItem,renameItem;
    public ImagePanel(Image image, int w, int h) {
        super();
        content = image;
        width  = w;
        height = h;
        popup = new JPopupMenu();
        rotateItem = new JMenuItem("Rotate...");
        renameItem = new JMenuItem("Rename...");
        popup.add(rotateItem);
        popup.add(renameItem);
        this.add(popup);
        this.addMouseListener(new MouseAdapter() {
            public void mouseReleased(MouseEvent me) {
                if ( me.isPopupTrigger() ) {
                    popup.show(me.getComponent(),me.getX(),me.getY());
                }
            }
        });
    }
    public void paintComponent(final Graphics g) {
        super.paintComponent(g);
        g.drawImage(content,0,0,width,height,null);
    }
    public Dimension getPreferredSize() {
        return new Dimension(width,height);
    }
}

 

[Freecastle]

Jetzt sag mir bitte nicht, dass so bei dir funktioniert, bei mir tut es das nämlich nicht

Das mit "mouseReleased" hatte ich sogar schon bevor ich es mit "mouseClicked" versucht habe, aber mit dem selben Ergebnis.

Ich hab hier mal noch nen Screenshot gemacht...
<< KLICK MICH >>

 

[Wildcard]

Versuch stattdessen mal ein JLabel mit ImageIcon, das ist weniger Fehleranfällig.

 

[André Uhres]

Jetzt sag mir bitte nicht, dass so bei dir funktioniert, bei mir tut es das nämlich nicht ..

Das funzt einwandfrei mit 1.5.0_08 bei mir.
Mit "mouseClicked" kommt allerdings gar kein Popup.

 

[FreeCastle]

Hm sry, ich bin irgendwie die ganze Zeit hier nicht ins Forum gekommen. Aber das lag dann wohl irgendwie an mir, da das wohl sonst bei allen geklappt hat.. na egal.

Also ich hab das jetzt stattdessen als Label mit nem ImageIcon gemacht und das funzt einwandfrei
Komisch ist es aber doch, dass das z.B. bei Andre so funktioniert und bei mir nicht. Ich habe übrigens auch das JDK 5, und wenn ichs recht in Erinnerung habe, sogar auch das Update 8...