Problem with findComponentAt highlighting wrong JLabel

Question

I've got a small 6 by 7 grid of jlabels that I would like to turn red if clicked on. In the japplet shown below, this works, sort of: it turns the label to its left red but not the label under the mouse. Any ideas on what I'm doing wrong? Thanks!
MyApplet.java
import javax.swing.JApplet;
public class MyApplet extends JApplet
    MyPane2 pane = new MyPane2();
    public void init()
        getContentPane().add(pane);
}MyPane2.java
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
class MyPane2 extends JPanel implements MouseListener
    private static final int MAX_ROW = 6;
    private static final int MAX_COL = 7;
    private String whiteDot = ".\\petes\\brief3\\whiteDot.jpg";
    private String redDot = ".\\petes\\brief3\\redDot.jpg";
    private JLabel[][] myLabelGrid;
    private JPanel gridPane = new JPanel();
    public MyPane2()
        super();
        add(createLabelGridPane());
    private JPanel createLabelGridPane()
        gridPane = new JPanel();
        GridLayout myGridLO = new GridLayout(MAX_ROW, MAX_COL);
        myGridLO.setHgap(5);
        myGridLO.setVgap(5);
        gridPane.setLayout(myGridLO);
        gridPane.addMouseListener(this);
        myLabelGrid = new JLabel[MAX_COL][MAX_ROW];
        for (int row = 0; row < MAX_ROW; row++)
            for (int col = 0; col < MAX_COL; col++)
                //myLabelGrid[col][row] = new JLabel(new ImageIcon(whiteDot));
                myLabelGrid[col][row] = new JLabel("W");
                gridPane.add(myLabelGrid[col][row]);
        return gridPane;
    public void mouseClicked(MouseEvent me)
        JPanel myPanel = (JPanel)me.getSource();
        System.out.println(findComponentAt(me.getX(), me.getY()));
        Component myComponent = findComponentAt(me.getX(), me.getY());
        if (myComponent instanceof JLabel)
            //((JLabel)myComponent).setIcon(new ImageIcon(redDot));
            ((JLabel)myComponent).setText("R");
        repaint();
    public void mouseEntered(MouseEvent arg0){}
    public void mouseExited(MouseEvent arg0){}
    public void mousePressed(MouseEvent arg0){}
    public void mouseReleased(MouseEvent arg0){}
}[edited to use a label with "W" or "R" rather than images]
Message was edited by:
petes1234

Answer 1

//  <applet code="FindComponentApplet" width="400" height="400"></applet>
//      No html file necessary, just type
//  appletviewer FindComponentApplet.java
//      at the prompt and press enter.
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
public class FindComponentApplet extends JApplet
    public void init()
        getContentPane().setLayout(new BorderLayout());
        getContentPane().add(new FindComponentPane());
class FindComponentPane extends JPanel implements MouseListener
    private static final int MAX_ROW = 6;
    private static final int MAX_COL = 7;
    Component lastSelected;
    public FindComponentPane()
        super(new BorderLayout());
        add(createLabelGridPane());
    private JPanel createLabelGridPane()
        JPanel gridPane = new JPanel(new GridLayout(MAX_ROW, MAX_COL, 5, 5));
        gridPane.addMouseListener(this);
        JLabel[][] myLabelGrid = new JLabel[MAX_COL][MAX_ROW];
        for (int row = 0; row < MAX_ROW; row++)
            for (int col = 0; col < MAX_COL; col++)
                JLabel label = new JLabel("W", JLabel.CENTER);
                label.setBorder(BorderFactory.createEtchedBorder());
                myLabelGrid[col][row] = label;
                gridPane.add(myLabelGrid[col][row]);
        return gridPane;
    public void mouseClicked(MouseEvent me)
        JPanel panel = (JPanel)me.getSource();
        Component component = panel.findComponentAt(me.getX(), me.getY());
        System.out.println(component.getClass().getName());
        if (component instanceof JLabel)
            if(lastSelected != null)
                ((JLabel)lastSelected).setForeground(Color.black);
            ((JLabel)component).setForeground(Color.red);
            // Since there are gaps among the labels we need this in here.
            lastSelected = component;
        repaint();
    public void mouseEntered(MouseEvent arg0){}
    public void mouseExited(MouseEvent arg0){}
    public void mousePressed(MouseEvent arg0){}
    public void mouseReleased(MouseEvent arg0){}
}