3

So the functionality I need is the Combobox which upon expansion can be right clicked and then a popup menu shows which shows different actions.

For that I extended the JCombobox with following constructor:

    public HistoryComboBox(DefaultComboBoxModel model) {
    super(model);
    super.setUI(new BasicComboBoxUI(this){

        @Override protected ComboPopup createPopup() {
            return new HistoryComboPopup(comboBox);
        }
    });

And created class HistoryComboPopup which extends BasicComboPopup, and has a mouselistener

@Override
protected MouseListener createListMouseListener() {
    if (handler2 == null)
        handler2 = new Handler2();
    return handler2;
}

private class Handler2 implements MouseListener {
    @Override
    public void mouseClicked(MouseEvent e) {
        JPopupMenu popup = new JPopupMenu();
        ...
        popup.show();
    }
}

The popup works, and I can handle its menu click. The only trouble is that the combobox closes when i open JPopupMenu and item on which i clicked right click is no longer seen.

I'm currently out of ideas what to do about this, so any help would greatly be appreciated.

EDIT: i mashed up a compileable example:

public class MainPanel extends JPanel{
public MainPanel() {
    super(new BorderLayout());

    JComboBox combo1 = makeComboBox(5);
    combo1.setUI(new BasicComboBoxUI() {
        @Override protected ComboPopup createPopup() {
            return new HistoryComboPopup(comboBox);
        }
    });

    add(combo1,BorderLayout.NORTH);
}

private static JComboBox makeComboBox(int size) {
    DefaultComboBoxModel model = new DefaultComboBoxModel();
    for(int i=0;i<size;i++) {
        model.addElement("No."+i);
    }
    return new JComboBox(model);
}

public static void main(String[] args) {
    EventQueue.invokeLater(new Runnable() {
        @Override public void run() {
            JFrame frame = new JFrame("DisableRightClick");
            frame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
            frame.getContentPane().add(new MainPanel());
            frame.pack();
            frame.setLocationRelativeTo(null);
            frame.setVisible(true);
        }
    });
}
}

class HistoryComboPopup extends BasicComboPopup {
private Handler2 handler2;

@Override
public void uninstallingUI() {
    super.uninstallingUI();
    handler2 = null;
}

public HistoryComboPopup(JComboBox combo) {
    super(combo);
}

@Override
protected MouseListener createListMouseListener() {
    if (handler2 == null)
        handler2 = new Handler2();
    return handler2;
}

private class Handler2 implements MouseListener {
    @Override public void mouseEntered(MouseEvent e) {}
    @Override public void mouseExited(MouseEvent e) {}
    @Override public void mouseClicked(MouseEvent e) {}
    @Override public void mousePressed(MouseEvent e) {}

    @Override
    public void mouseReleased(MouseEvent e) {
        if (SwingUtilities.isRightMouseButton(e)) {
            JPopupMenu popup = new JPopupMenu();

            JMenuItem copymethod = new JMenuItem("copy");
            copymethod.addMouseListener(new MouseAdapter() {

                @Override
                public void mousePressed(MouseEvent e) {
                    System.out.println("copy clicked");
                }
            });
            popup.add(copymethod);
            popup.show(HistoryComboPopup.this, 
                    e.getXOnScreen() - HistoryComboPopup.this.getLocationOnScreen().x, 
                    e.getYOnScreen() - HistoryComboPopup.this.getLocationOnScreen().y);

        } else {
            comboBox.setSelectedIndex(list.getSelectedIndex());
            comboBox.setPopupVisible(false);
        }
    }
}
}
2

1 Answer 1

1

So the functionality I need is the Combobox which upon expansion can be right clicked and then a popup menu shows which shows different actions.

  • popup.show(); isn't proper accelator

  • that not possible directly, its too hard to override common bug,

  • Swing doesn't allows to shows two lightwieght popup components in same time

  • use JWindows or un-decorated JDialog instead of JPopup

code example demonstrated an official bug

    import javax.swing.*;
    import java.awt.event.*;
    
    public class Test {
    
        public static void main(String[] args) {
            JFrame frame = new JFrame();
            frame.setSize(400, 400);
            frame.setVisible(true);
            String[] list = {"1", "2", "3", "4",};
            JComboBox comb = new JComboBox(list);
            final JPopupMenu pop = new JPopupMenu();
            pop.add(comb);
            frame.addMouseListener(new MouseAdapter() {
    
                @Override
                public void mousePressed(MouseEvent e) {
                    System.out.println("mousePressed");
                    pop.show(e.getComponent(), e.getX(), e.getY());
                }
            });
        }
   }
4
  • 1
    the bug is a) unrelated (as far as I understand the question, slightly confused though :-) it's about adding a comboBox to a popup, the question is about showing another popup on clicking an item in the combo's popup b) closed as not-reproducible since jdk5, is it still there?
    – kleopatra
    Jul 9, 2013 at 14:00
  • doesn't matter if we talking about choice a or b, in both cases is this bug still here, one of popup go away,
    – mKorbel
    Jul 9, 2013 at 14:55
  • that's not a bug as such (and the issue you reference doesn't complain about that but about a NPE in the bowels of BasicComboPopup) - popup's are designed to behave exactly as they do (Es kann nur einen geben :-)
    – kleopatra
    Jul 9, 2013 at 15:00
  • Ich weiss nichts uber, I have some post and with comments about JPopup.isHeavy... , can't found this thread, not sure about mixing Swing and AWT, but AFAIK AWT Popup nothing changed
    – mKorbel
    Jul 9, 2013 at 18:41

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.