Images do not display correctly

[celine mcgowan]

Hi Everyone,
I'm a bit of a newbie & i'm trying to implement an LED display. I have 4 .gif's to represent the 'bars' of an LED - vertical & horizontal, lit & unlit. I've started by trying to organise the 7 'bars' in a GridLayout , but when i run this code, i just get two of the bars displaying in the window - one Horizontal & one Vertical....

  
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
public class LEDtest extends JPanel{
    JLabel horizl;
    JLabel horizu;
    JLabel vertl;
    JLabel vertu;
    
    public LEDtest(){
        horizl = new JLabel(new ImageIcon("horiz_lit.gif"));
        horizu = new JLabel(new ImageIcon("horiz_unlit.gif"));
        vertl = new JLabel(new ImageIcon("vertical_lit.gif"));
        vertu = new JLabel(new ImageIcon("vertical_unlit.gif"));
        
        JPanel ledPanel = new JPanel();
        ledPanel.setLayout(new GridLayout(1,2));
        ledPanel.add(horizl);
        ledPanel.add(vertl);
        ledPanel.add(horizl);
        ledPanel.add(vertl);
        ledPanel.add(horizl);
        ledPanel.add(vertl);
        ledPanel.add(vertl);
        
        setLayout(new BorderLayout());
        add(ledPanel);
        //setBorder(BorderFactory.createEmptyBorder(20,20,20,20));
    }
    public static void main (String[] args){
        JFrame frame = new JFrame("LEDTest");
        frame.addWindowListener(new WindowAdapter(){
            public void windowClosing(WindowEvent e){
                System.exit(0);
            }
        });
    
        frame.setContentPane(new LEDtest());
        frame.pack();
        frame.setVisible(true);
    }
}

Can anyone tell me where i'm going wrong? I'd really appreciate any help at all.
Thanks a million,
Celine

[Joe Ess]

Originally posted by celine mcgowan:

        ledPanel.setLayout(new GridLayout(1,2));        
  

For starters, your GridLayout has one row and two columns. That gives you two slots to display components in.

[celine mcgowan]

Hi Joe,
Thanks a million for your help. I actually got this by creating two panels with border layouts & making two 'squares' from them. I then made another panel and added the two squares to it, so i have a nice 'figure of eight' pattern on the screen.
Thanks again!
Celine