one key press

[chen young]

Hi,

I created a JFrame that contain JComboBox and JButton.
When the user will press ENTER action event should run , the problem is it happen only after the second press on the ENTER since the first press will remark the text in the JComboBox and the second will activate the button action event.

How can I do it with one press ?

Thanks

[Maneesh Godbole]

If the action you want to run is the same which should be triggered on the click of the button, check out JFrame#getRootPane#setDefaultButton(JButton button)

[chen young]

I already try this option:

private void initGUI() {
		getContentPane().setBackground(new java.awt.Color(192, 192, 192));

		jPanel = new JPanel();
		getContentPane().add(jPanel, BorderLayout.CENTER);
		jPanel.setBorder(BorderFactory.createEtchedBorder(BevelBorder.RAISED));
		DesignGridLayout designGridLayout = new DesignGridLayout(jPanel);

		jLabel = new JLabel();
		jLabel.setText("Find What:");

		findBox = new JComboBox();
		findBox.setEditable(true);
		findBox.getEditor().getEditorComponent().addKeyListener(this);

		findNextButton = new JButton("Find Next");
		findNextButton.addActionListener(this);
		findNextButton.addKeyListener(this);
		this.getRootPane().setDefaultButton(findNextButton);

		designGridLayout.row().grid(jLabel).add(findBox, 2).add(findNextButton, 1);
		setSize(400, 110);
	} 

    
public void actionPerformed(ActionEvent actionEvent) {
		if (actionEvent.getActionCommand().equals("Find Next")) {
....
..
. 
}

 
public void keyReleased(KeyEvent keyEvent) {
		if (!findBox.getEditor().getItem().equals("")) {
			findNextButton.setEnabled(true);
		} else {
			findNextButton.setEnabled(false);
		}
	}

[Maneesh Godbole]

If you click the button with the mouse, does the required action take place?

[chen young]

yes

[Maneesh Godbole]

Can you try, after making it the last line of your constructor?
Just noticed you are having an action listener as well as a key listener. Why is that? If you are effectively calling the same method as defined in action performed,please take out the key listener. You don't need it.

[chen young]

Do you meant to put this line :

  this.getRootPane().setDefaultButton(findNextButton);   

at the end of constructor ?


I add the KeyListener so I could disable/enable the button according to the text in the textfield

  
public void keyReleased(final KeyEvent keyEvent) {
		if (!findBox.getEditor().getItem().equals("")) {
			findNextButton.setEnabled(true);
		} else {
			findNextButton.setEnabled(false);
		}
	}

[Maneesh Godbole]

chen yan wrote:Do you meant to put this line :

  this.getRootPane().setDefaultButton(findNextButton);   

at the end of constructor ?

Yes. That is what I meant.
You have also added the key listener to the button!

  findNextButton.addKeyListener(this);

[chen young]

I delete this line the Key Listener and change the position but still I need to press twice on the ENTER (first press will mark the text in the field and the second press will activate the button action event).

[Maneesh Godbole]

If the combo popup is showing, there is no way you can trigger the action for your button.
Anybody else?

[Michael Dunn]

works OK using an AWTEventListener

    
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
class Testing
{

  public void buildGUI()
  {
    JComboBox cbo = new JComboBox(new String[]{"London","Madrid","New York","Rome","Sydney","Washington"});
    cbo.setEditable(true);
    final JButton btn = new JButton("OK");
    JPanel p = new JPanel();
    p.add(cbo); p.add(btn);
    JFrame f = new JFrame();
    f.getContentPane().add(p);
    f.setSize(300,200);
    f.setLocationRelativeTo(null);
    f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    f.setVisible(true);
    btn.addActionListener(new ActionListener(){
      public void actionPerformed(ActionEvent ae){
        System.out.println("OK");
      }
    });
    Toolkit.getDefaultToolkit().addAWTEventListener( new AWTEventListener(){
      public void eventDispatched(AWTEvent e)
      {
        if(e.getID() == KeyEvent.KEY_PRESSED)
        {
          if(((KeyEvent)e).getKeyCode() == KeyEvent.VK_ENTER) btn.doClick();
        }
      }
    }, AWTEvent.KEY_EVENT_MASK);
  }
  public static void main(String[] args)
  {
    SwingUtilities.invokeLater(new Runnable(){
      public void run(){
        new Testing().buildGUI();
      }
    });
  }
}