JMenuItem und Accelerator

[Gast2]

Hallo zusammen,

kann man ein Accelerator immer verfügbar machen, auch wenn ein JDialog usw. gerade den Focus hat?

[HIGHLIGHT="Java"]
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.InputEvent;
import java.awt.event.KeyEvent;

import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.KeyStroke;
import javax.swing.SwingUtilities;

public class TestMenue extends JFrame {

JMenuBar menuBar;

public TestMenue() {
menuBar = new JMenuBar();

JMenu test = new JMenu("test");

JMenuItem oeffnen = new JMenuItem("Öffnen");
oeffnen.addActionListener(new ActionListener() {

public void actionPerformed(ActionEvent e) {
JDialog dialog = new JDialog();
dialog.setSize(100, 100);
dialog.setVisible(true);
}

});
JMenuItem save = new JMenuItem("Save");
save.addActionListener(new ActionListener() {

public void actionPerformed(ActionEvent e) {
System.out.println("Save soll immer funktionieren");

}

});
save.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_S, InputEvent.CTRL_DOWN_MASK));
test.add(oeffnen);
test.add(save);
menuBar.add(test);
setJMenuBar(menuBar);
setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
setSize(400, 400);

}

public static void main(String[] args) {
SwingUtilities.invokeLater(new Runnable() {

public void run() {
new TestMenue().setVisible(true);

}

});

}

}


[/HIGHLIGHT]

Also wenn man auf öffnen klickt dann funktioniert STRG + S nicht mehr wie kann man das ändern? Nur über die InputMap??

 

[Ebenius]

Ich glaub, das kann man gar nicht einfach so ändern. Die Accelerators hängen in der WHEN_IN_FOCUSED_WINDOW-InputMap. Wenn ein anderes Fenster den Fokus hat, dann greifen diese key mappings natürlich nicht.

Ebenius

 

[Gast2]

jop hab ich auch gemerkt...
Es gibt 2 Möglichkeiten den JDialog usw. den frame als owner mitgeben
oder
[HIGHLIGHT="Java"]

KeyboardFocusManager.getCurrentKeyboardFocusManager().addKeyEventPostProcessor(new KeyEventPostProcessor(){

@Override
public boolean postProcessKeyEvent(KeyEvent e) {
// TODO Auto-generated method stub
return false;
}

});
[/HIGHLIGHT]

 

[Gast2]

okay ich hab es nochmal versucht auch wenn der JDialog einen owner hat funktioniert es nicht man muss noch einen JButton reinsetzen...
Wie könnte man das Problem umgehen(außer variante 2 )... Kennt noch jemand eine Möglichkeit
[HIGHLIGHT="Java"]
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.InputEvent;
import java.awt.event.KeyEvent;

import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.KeyStroke;
import javax.swing.SwingUtilities;

public class TestMenue extends JFrame {

JMenuBar menuBar;

public TestMenue() {
menuBar = new JMenuBar();

JMenu test = new JMenu("test");

JMenuItem oeffnen = new JMenuItem("Öffnen");
oeffnen.addActionListener(new ActionListener() {

public void actionPerformed(ActionEvent e) {
JDialog dialog = new JDialog(TestMenue .this);
dialog .add(newJButton());
dialog.setSize(100, 100);
dialog.setVisible(true);
}

});
JMenuItem save = new JMenuItem("Save");
save.addActionListener(new ActionListener() {

public void actionPerformed(ActionEvent e) {
System.out.println("Save soll immer funktionieren");

}

});
save.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_S, InputEvent.CTRL_DOWN_MASK));
test.add(oeffnen);
test.add(save);
menuBar.add(test);
setJMenuBar(menuBar);
setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
setSize(400, 400);

}

public static void main(String[] args) {
SwingUtilities.invokeLater(new Runnable() {

public void run() {
new TestMenue().setVisible(true);

}

});

}

}

[/HIGHLIGHT]

 

[Ebenius]

Du brauchst nen Fokus in dem Dialog. Wenn keine Komponente den Fokus bekommen kann, wird's schwer. Vielleicht geht "dialog.setFocusable(true);" statt des JButtons.

Viel Erfolg!
Ebenius

 

[Gast2]

nee ich hab mich glaub falsch ausgedrückt...
Ich dachte wenn man einem jDialog einen owner mitgibt dann funktionieren die KeyEvent im Menü noch, aber sobald der JDialog einen Komponente enthält funktionieren sie nicht mehr--> das ist mir aufgefallen.
Meine Frage war ob es vielleicht eine andere Möglichkeit wie
[HIGHLIGHT="Java"]
KeyboardFocusManager.getCurrentKeyboardFocusManager().addKeyEventPostProcessor(new KeyEventPostProcessor() {

public boolean postProcessKeyEvent(KeyEvent e) {

if (e.isControlDown() && KeyStroke.getKeyStrokeForEvent(e).equals(menuItem.getAccelerator())) {
actionListener.actionPerformed(new ActionEvent(menuItem, e.getID(), "KeyEvent.KEY_TYPED"));
}
return false;
}
});

[/HIGHLIGHT]
wie sowas hier gibt? das funktioniert auf jeden Fall weiß aber nicht ob es sehr performant ist =)...

 

[Ebenius]

Probier mal, ob sowas geht: [HIGHLIGHT=Java]final KeyboardFocusManager fm =
KeyboardFocusManager.getCurrentKeyboardFocusManager();
fm.addKeyEventPostProcessor(new KeyEventPostProcessor() {

public boolean postProcessKeyEvent(KeyEvent e) {
if (fm.getActiveWindow() != mainFrame
&& !e.isConsumed()
&& isValidKeyEventForKeyBindings(e)) {
SwingUtilities.processKeyBindings(new KeyEvent(mainFrame
.getRootPane(), e.getID(), e.getWhen(), e.getModifiersEx(), e
.getKeyCode(), e.getKeyChar(), e.getKeyLocation()));
}
return false;
}

// copied from SwingUtitilities (invisible)
private boolean isValidKeyEventForKeyBindings(KeyEvent e) {
if (e.getID() == KeyEvent.KEY_TYPED) {
int mod = e.getModifiers();
if (((mod & ActionEvent.ALT_MASK) != 0)
&& ((mod & ActionEvent.CTRL_MASK) == 0)) {
// filter out typed "alt-?" keys, but not those created
// with AltGr, and not control characters
return false;
}
}
return true;
}
});[/HIGHLIGHT]

Ebenius

 

[Gast2]

Das geht ist aber langsamer wie meine variante...

 

[Ebenius]

als

Stimmt. Aber Performance spielt hier keine Rolle. Wieviele Tastenanschläge möchtest Du denn pro Millisekunde verabeiten? Meine Lösung schlägt einfach eine Brücke zu allen WHEN_IN_FOCUSED_WINDOW-KeyBindings. Wenn Du das nur für eine einzige Aktion benötigst, dann sind beide Ansätze natürlich übertrieben.

Ebenius

Ebenius

 

[Gast2]

Ja aber was an deiner besser ist, es werden die Tastenschläge(Shortcuts) dann auf jeden Fall nur einmal ausgeführt...
Nee ist geplant für alle shortcuts, die in einem JMenu vorkommen...

 

[Gast2]

[HIGHLIGHT="Java"]
KeyStroke.getKeyStrokeForEvent(e).equals(menuItem.getAccelerator()
[/HIGHLIGHT]

die Zeile muss noch mit rein sonst führt er es bei jedem Tastenschlag aus und führt die aktion gefühlte 100 mal aus

 

[Ebenius]

[HIGHLIGHT="Java"]
KeyStroke.getKeyStrokeForEvent(e).equals(menuItem.getAccelerator()
[/HIGHLIGHT]

die Zeile muss noch mit rein sonst führt er es bei jedem Tastenschlag aus und führt die aktion gefühlte 100 mal aus

Es war von mir beabsichtigt, alle in Frage kommenden KeyBindings an das mainFrame weiterzuleiten. Performance ist dabei kein Thema.

Ebenius

 

[Gast2]

Ja du hast alle weitergeleitet, was im Prinzip auch okay ist, aber diese werden nicht nur einmal weitergeleitet sondern mehrmals

Sagen wir z.B. du hast einen shortcut STRG + G um irgendwas zu zoomen...
Dann wird dieser merhmals ausgeführt und die zoom methode wird mehrmals ausgeführt...
Das heißt es wird nicht ein zoom step ausgeführt sondern mehrere auf einmal

 

[Ebenius]

Ja du hast alle weitergeleitet, was im Prinzip auch okay ist, aber diese werden nicht nur einmal weitergeleitet sondern mehrmals

Nö. Da musst Du was anderes falsch machen. Hier ein bisschen Schweine-Code zum Testen für Dich: [HIGHLIGHT="Java"]public static void main(String[] args) {
final JTextPane textPane = new JTextPane();
textPane.setPreferredSize(new Dimension(400, 400));

final JPanel buttonPanel = new JPanel();
buttonPanel.add(new JButton(new AbstractAction("Open Dialog") {

{
putValue(MNEMONIC_KEY, new Integer(KeyEvent.VK_O));
}

public void actionPerformed(ActionEvent e) {
final JComponent c = (JComponent) e.getSource();
final JFrame mainFrame = (JFrame) SwingUtilities.getWindowAncestor(c);
final JDialog dialog = new JDialog(mainFrame, "Child Dialog");
final JLabel label =
new JLabel("Try the main frame's menu action accelerator here");
final JPanel dialogPane = new JPanel(new GridLayout(2, 1, 6, 6));
dialogPane.add(label);
dialogPane.add(new JButton(new AbstractAction("Close") {

public void actionPerformed(ActionEvent e) {
dialog.dispose();
}
}));
dialog.setContentPane(dialogPane);
dialog.setModal(true);
dialog.pack();
dialog.setResizable(false);
dialog.setLocationRelativeTo(mainFrame);

final KeyboardFocusManager fm =
KeyboardFocusManager.getCurrentKeyboardFocusManager();
final KeyEventPostProcessor keyEventPostProcessor =
new KeyEventPostProcessor() {

public boolean postProcessKeyEvent(KeyEvent e) {
if (fm.getActiveWindow() != mainFrame
&& !e.isConsumed()
&& isValidKeyEventForKeyBindings(e)) {
SwingUtilities.processKeyBindings(new KeyEvent(mainFrame
.getRootPane(), e.getID(), e.getWhen(), e
.getModifiersEx(), e.getKeyCode(), e.getKeyChar(),
e.getKeyLocation()));
}

return false;
}

// copied from SwingUtitilities (invisible)
private boolean isValidKeyEventForKeyBindings(KeyEvent e) {
if (e.getID() == KeyEvent.KEY_TYPED) {
int mod = e.getModifiers();
if (((mod & ActionEvent.ALT_MASK) != 0)
&& ((mod & ActionEvent.CTRL_MASK) == 0)) {
// filter out typed "alt-?" keys, but not those created
// with AltGr, and not control characters
return false;
}
}
return true;
}
};

try {
fm.addKeyEventPostProcessor(keyEventPostProcessor);
dialog.setVisible(true);
} finally {
fm.removeKeyEventPostProcessor(keyEventPostProcessor);
}
}
}), BorderLayout.PAGE_END);

final JPanel panel = new JPanel(new BorderLayout(6, 6));
panel.add(buttonPanel, BorderLayout.LINE_END);
panel.add(new JScrollPane(textPane), BorderLayout.CENTER);

final JMenuBar menuBar = new JMenuBar();
final JMenu menu = new JMenu("Menu");
menu.setMnemonic(KeyEvent.VK_M);
menu.add(new AbstractAction("Add some Text") {

{
putValue(MNEMONIC_KEY, new Integer(KeyEvent.VK_A));
putValue(ACCELERATOR_KEY, KeyStroke.getKeyStroke(KeyEvent.VK_ENTER,
InputEvent.CTRL_DOWN_MASK));
}

public void actionPerformed(ActionEvent e) {
final Document doc = textPane.getDocument();
try {
doc.insertString(doc.getLength(), new Date()
+ " Action performed\n", null);
} catch (BadLocationException ex) {
ex.printStackTrace();
}
}
});
menuBar.add(menu);

final JFrame mainFrame = new JFrame("KeyBinding Fun");
mainFrame.setJMenuBar(menuBar);
mainFrame.setContentPane(panel);
mainFrame.pack();
mainFrame.setLocationRelativeTo(null);
mainFrame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
mainFrame.setVisible(true);
}[/HIGHLIGHT]

Ebenius

 

[Gast2]

ahhhhhhhhhhhhhhhhh =)... ich hab dem KeyManager zuviele gleiche KeyEventPostProcessor hinzugefügt, darum hat er es zu oft ausgeführt *omg*...
danke hast mich auf den richtigen weg gebracht ...