Popup Menu

[Pranay Narotam]

Hi all

I have a JTable, and if a user "right clicks" on a row, a pop up menu should come up.

This is so far what I have done, but its not working correctly.

 public void mouseClicked(MouseEvent evt)
{
if (evt.getButton() == 3)
{
popUp();
}
} 

 public void popUp()
{
jpMenu = new JPopupMenu();
item1 = new JMenuItem("ITEM 1");
item2 = new JMenuItem("ITEM 2");
jpMenu.add(item1);
jpMenu.add(item2);
add(jpMenu);
jpMenu.setVisible(true);
} 

The pop up menu does come up when I "right click", however the screen of the table disappears and the
popup menu comes up on the corner of the screen, i want it to be where the user "right clicks"

Your help will be greatly appreciated

Pranay

[Rob Spoor]

 

Pranay Narotam wrote:

if (evt.getButton() == 3)

Please use MouseEvent.BUTTON3; it will make reading a bit easier, and if Sun decides to change its value from 3 to something else like -3, your code will still work.

jpMenu = new JPopupMenu();
...
add(jpMenu);
jpMenu.setVisible(true);

You're using a BorderLayout, which allows only one component at its center. By using add(jpMenu) you are replacing the table with the popup menu.

This also isn't the way popup menus work. They should not get added to an existing control; instead, they have their own window which you can show as follows:

jpMenu = new JPopupMenu();
...
jpMenu.show(parent, x, y);

Combined with your mouse listener:

public void mouseClicked(MouseEvent evt)
{
if (evt.getButton() == MouseEvent.BUTTON3)
{
Component source = (Component)evt.getSource();
int x = evt.getX();
int y = evt.getY();
popUp(source, x, y);
}
}
public void popUp(Component invoker, int x, int y)
{
jpMenu = new JPopupMenu();
item1 = new JMenuItem("ITEM 1");
item2 = new JMenuItem("ITEM 2");
jpMenu.add(item1);
jpMenu.add(item2);
jpMenu.show(invoker, x, y);
}

[Pranay Narotam]

Hi

Thanks alot, It works perfectly



Pranay

[Pranay Narotam]

Hi again

I am looking at the code, it works perfectly but i don't seem to understand why there is a parameter "COMPONENT" in the method

public void popUp(Component invoker, int x, int y) 

Thanks again

Pranay

[Rob Spoor]

You need to show the popup for a specific component. I've decided to get that from the mouse event, but you can hard code it to be your table as well of course. In that case, you can remove the entire parameter:

public void mouseClicked(MouseEvent evt)
{
if (evt.getButton() == MouseEvent.BUTTON3)
{
int x = evt.getX();
int y = evt.getY();
popUp(x, y);
}
}
public void popUp(int x, int y)
{
jpMenu = new JPopupMenu();
item1 = new JMenuItem("ITEM 1");
item2 = new JMenuItem("ITEM 2");
jpMenu.add(item1);
jpMenu.add(item2);
jpMenu.show(jTable, x, y); // show only for your JTable
}