CODEWITHSUNDEEP
javaswingjtextareakey-bindingskeystrokes
Karlovsky120
Karlovsky120

Reputation: 6352

key-bindings and holding down keys

I have created a key binding for a JTextArea Component. When invoked, it creates a new instance of itself and sets focus to it.

If you hold down the enter (which invokes key binding) my program will start spitting out bunch of JTextArea instances.

Is there a way to force the user to press enter againg to create a new instance?

Do I have I switch to KeyListeners or is there a way with key bindings?

Upvotes: 3

Views: 2445

Answers (3)

Venryx
Venryx

Reputation: 17999

Here is the code I use, to have an action only run when a key is first pressed down:

private void registerKeyBindings(final JFrame frame) {
    var inputMap = frame.getRootPane().getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW);
    inputMap.put(KeyStroke.getKeyStroke(KeyCode.G.getInputEventCode(), 0, false), "g_down");
    inputMap.put(KeyStroke.getKeyStroke(KeyCode.G.getInputEventCode(), 0, true), "g_up");
    
    frame.getRootPane().getActionMap().put("g_down", new AbstractAction() {
      @Override public void actionPerformed(ActionEvent e) {
        if (gDown) return;
        gDown = true;

        // put your custom key-down-action code here
      }
    });
    frame.getRootPane().getActionMap().put("g_up", new AbstractAction() {
      @Override public void actionPerformed(ActionEvent e) {
        gDown = false;
      }
    });
}
Boolean gDown = false;

Upvotes: 0

kleopatra
kleopatra

Reputation: 51525

the way to do it with keybindings is to have two actions:

  • an action creating the component is bound to the pressed enter, it disables itself when inserting the component
  • an action enabling the action again is bound to the released enter

Some code:

// the action to create the component
public static class CreateAction extends AbstractAction {

    private Container parent;
    private Action enableAction;

    public CreateAction(Container parent) {
        this.parent = parent;
        enableAction = new EnableAction(this);
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        setEnabled(false);
        Component field = createTextField();
        parent.add(field);
        parent.revalidate();
        field.requestFocus();
    }

    int count;
    private Component createTextField() {
        // just for fun counting the fields we create
        JTextField field = new JTextField("field: " + count++, 20);
        field.getInputMap().put(KeyStroke.getKeyStroke("ENTER"), 
                "createComponent");
        field.getActionMap().put("createComponent", this);
        field.getInputMap().put(KeyStroke.getKeyStroke("released ENTER"), 
                "enableCreation");
        field.getActionMap().put("enableCreation", enableAction);
        return field;
    }

}

// the action that enables another
public static class EnableAction extends AbstractAction {

    Action toEnable;

    public EnableAction(Action toEnable) {
        this.toEnable = toEnable;
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        toEnable.setEnabled(true);
    }

}

// usage
final JComponent parent = new JPanel(new MigLayout("wrap"));
// here I'm lazy and let the action create the very first component as well
add.actionPerformed(null);
add.setEnabled(true);

Note that the same instances of the actions are registered to all components, so it doesn't matter which has the focus (and ultimately enables the creation again)

Upvotes: 2

MadProgrammer
MadProgrammer

Reputation: 347204

You specify that a KeyStroke only fire on key release when you're setting up the input map

See KeyStroke getKeyStroke(int keyCode, int modifiers, boolean onKeyRelease)

Upvotes: 4

Related Questions