Opening up a new frame

Question

Hi everyone, im new here.
I'm creating a friendly GUI project with buttons and detecting key press events. Currently, I'm having problems detecting keypress for a new frame(sub frame) that i've opened.
I've 2 AWTEventListener, 1 for main frame and 1 for sub frame.
Is there any ways that i can clear this problem?
Thanks in advance

Answer 1

Hi camickr and MikeP.
I tried using key binders for the main frame. I'm starting with button key press down. However, the focus doesn't seem to shift over to the next button, and the program will stay there.
But when i use System.out.println("Button focused on " +current); without the request focus line.
It managed to show which button is suppose to have the focus.
Below is the edited program with key binders
import javax.swing.AbstractAction;
import javax.swing.Action;
import javax.swing.KeyStroke;
import javax.swing.SwingUtilities;
import javax.swing.JPanel;
import javax.swing.JFrame;
import java.awt.GridBagLayout;
import javax.swing.JButton;
import java.awt.GridBagConstraints;
import java.awt.Insets;
import java.awt.event.ActionEvent;
public class MainFrame extends JFrame{
     private static final long serialVersionUID = 1L;
     private JPanel jContentPane = null;
     public JButton[] MenuButton = new JButton[8];
          for (int i = 0; i<8 ; i++)
               MenuButton[i] = null;
     private int current = 0;
     private JPanel jContentPane() {
          if (jContentPane == null) {
               GridBagConstraints gridBagConstraints7 = new GridBagConstraints();
               gridBagConstraints7.gridx = 1;
               gridBagConstraints7.insets = new Insets(8, 8, 8, 8);
               gridBagConstraints7.fill = GridBagConstraints.BOTH;
               gridBagConstraints7.gridy = 3;
               GridBagConstraints gridBagConstraints6 = new GridBagConstraints();
               gridBagConstraints6.gridx = 1;
               gridBagConstraints6.insets = new Insets(8, 8, 8, 8);
               gridBagConstraints6.fill = GridBagConstraints.BOTH;
               gridBagConstraints6.gridy = 2;
               GridBagConstraints gridBagConstraints5 = new GridBagConstraints();
               gridBagConstraints5.gridx = 1;
               gridBagConstraints5.insets = new Insets(8, 8, 8, 8);
               gridBagConstraints5.fill = GridBagConstraints.BOTH;
               gridBagConstraints5.gridy = 1;
               GridBagConstraints gridBagConstraints4 = new GridBagConstraints();
               gridBagConstraints4.gridx = 1;
               gridBagConstraints4.insets = new Insets(8, 8, 8, 8);
               gridBagConstraints4.fill = GridBagConstraints.BOTH;
               gridBagConstraints4.gridy = 0;
               GridBagConstraints gridBagConstraints3 = new GridBagConstraints();
               gridBagConstraints3.gridx = 0;
               gridBagConstraints3.insets = new Insets(8, 8, 8, 8);
               gridBagConstraints3.fill = GridBagConstraints.BOTH;
               gridBagConstraints3.gridy = 3;
               GridBagConstraints gridBagConstraints2 = new GridBagConstraints();
               gridBagConstraints2.gridx = 0;
               gridBagConstraints2.insets = new Insets(8, 8, 8, 8);
               gridBagConstraints2.fill = GridBagConstraints.BOTH;
               gridBagConstraints2.gridy = 2;
               GridBagConstraints gridBagConstraints1 = new GridBagConstraints();
               gridBagConstraints1.gridx = 0;
               gridBagConstraints1.insets = new Insets(8, 8, 8, 8);
               gridBagConstraints1.fill = GridBagConstraints.BOTH;
               gridBagConstraints1.gridy = 1;
               GridBagConstraints gridBagConstraints = new GridBagConstraints();
               gridBagConstraints.gridx = 0;
               gridBagConstraints.fill = GridBagConstraints.BOTH;
               gridBagConstraints.insets = new Insets(8, 8, 8, 8);
               gridBagConstraints.gridy = 0;
               jContentPane = new JPanel();
               jContentPane.setLayout(new GridBagLayout());
               jContentPane.add(getJButton(), gridBagConstraints);
               jContentPane.add(getJButton1(), gridBagConstraints1);
               jContentPane.add(getJButton2(), gridBagConstraints2);
               jContentPane.add(getJButton3(), gridBagConstraints3);
               jContentPane.add(getJButton4(), gridBagConstraints4);
               jContentPane.add(getJButton5(), gridBagConstraints5);
               jContentPane.add(getJButton6(), gridBagConstraints6);
               jContentPane.add(getJButton7(), gridBagConstraints7);
               jContentPane.getInputMap().put(KeyStroke.getKeyStroke("DOWN"), "MoveDown");
               Action ActionDown = new AbstractAction() {
                   public void actionPerformed(ActionEvent e) {
                        current = current < 7 ? current = current + 1 : (current + 1 > 7 ? 0:1);
                      //System.out.println("press down, Button"+current+" on focus");
                      MenuButton[current].requestFocus();  //next button obtains focus
               jContentPane.getActionMap().put("MoveDown", ActionDown);
          return jContentPane;
     private JButton getJButton() {
          if (MenuButton[0] == null) {
               MenuButton[0] = new JButton();
               MenuButton[0].setText("Singapore");
          return MenuButton[0];
     private JButton getJButton1() {
          if (MenuButton[1] == null) {
               MenuButton[1] = new JButton();
               MenuButton[1].setText("World");
          return MenuButton[1];
     private JButton getJButton2() {
          if (MenuButton[2] == null) {
               MenuButton[2] = new JButton();
               MenuButton[2].setText("Business/Finance");
          return MenuButton[2];
     private JButton getJButton3() {
          if (MenuButton[3] == null) {
               MenuButton[3] = new JButton();
               MenuButton[3].setText("Sports");
          return MenuButton[3];
     private JButton getJButton4() {
          if (MenuButton[4] == null) {
               MenuButton[4] = new JButton();
               MenuButton[4].setText("Technology");
          return MenuButton[4];
     private JButton getJButton5() {
          if (MenuButton[5] == null) {
               MenuButton[5] = new JButton();
               MenuButton[5].setText("Entertainment");
          return MenuButton[5];
     private JButton getJButton6() {
          if (MenuButton[6] == null) {
               MenuButton[6] = new JButton();
               MenuButton[6].setText("Special Reports");
          return MenuButton[6];
     private JButton getJButton7() {
          if (MenuButton[7] == null) {
               MenuButton[7] = new JButton();
               MenuButton[7].setText("Travel");
          return MenuButton[7];
     public static void main(String[] args) {
          SwingUtilities.invokeLater(new Runnable() {
               public void run() {
                    MainFrame thisClass = new MainFrame();
                    thisClass.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                    thisClass.setVisible(true);
     public MainFrame() {
          super();
          initialize();
     private void initialize() {
          this.setSize(1024, 768);
          this.setContentPane(jContentPane());
          this.setTitle("Daily News");          
}Thanks for your time and help