JLabel appears as a JButton

[Gregory Ball]

What ever I try the Jlabel want appeear. Instead it comes up as one of the JButtons.
The code is a little messy, sos.
Any Help would be great

 
private static final long serialVersionUID = 1L;
	ImageIcon backImg;
	JLabel title;
	JButton btnNew;
    JButton btnLoad;
    JButton btnIns;
    JButton btnScore;
    JButton btnExit;
    
    
    public WelcomePage()
    {
        setLayout(null);
        title = new JLabel("Software for SETI", JLabel.CENTER);  
        title.setFont(new Font("SansSerif", Font.ITALIC | Font.BOLD, 90));
       
        title.setBorder(BorderFactory.createEmptyBorder(5, 5, 5, 5));
        title.setForeground(Color.white);
        title.setBounds(100, 110, 220, 35);
        add(title);
        
        backImg = new ImageIcon("Images/Welcome.jpg");
        btnNew = new JButton("New Game");
        btnLoad = new JButton("Load Game");
        btnIns = new JButton("Instructions");
        btnScore = new JButton("Score Board");
        btnExit = new JButton("Exit");
      
        
        add(btnNew);
        add(btnLoad);
        add(btnIns);
        add(btnScore);
        add(btnExit);
        
        btnNew.setBounds(225,240,220,35);
        btnLoad.setBounds(225,280,220,35);
        btnIns.setBounds(225,320,220,35);
        btnScore.setBounds(225,360,220,35);
        btnExit.setBounds(225,440,220,35);
       
        btnNew.addActionListener(this);
        btnLoad.addActionListener(this);
        btnIns.addActionListener(this);
        btnScore.addActionListener(this);
        btnExit.addActionListener(this);
    }
    
    public void actionPerformed(ActionEvent ae)
    {
        MainPanel panel=(MainPanel)getParent();
        if (btnNew.equals(ae.getSource()))
        {
            panel.hideAll();
          //  panel.pageName.setVisible(true);
        }
        else if (btnLoad.equals(ae.getSource()))
        {
            panel.hideAll();
         //   panel.pageLoad.setContent();
           // panel.pageLoad.setVisible(true);
        }
        else if (btnIns.equals(ae.getSource()))
        {
            panel.hideAll();
           // panel.pageIns.setVisible(true);
        }
        else if (btnScore.equals(ae.getSource()))
        {
            panel.hideAll();
          //  panel.pageScore.setContent();
           // panel.pageScore.setVisible(true);
        }
 
        else if (btnExit.equals(ae.getSource()))
        {
            System.exit(0);
        }
    }

    public void paint(Graphics g)
    {
    	title.repaint();
        backImg.paintIcon(this,g,0,0);
        btnNew.repaint();
        btnLoad.repaint();
        btnIns.repaint();
        btnScore.repaint();

        btnExit.repaint();
        
    }
}
  

[Rob Spoor]

Please Use Code Tags.

[pete stein]

I think that most of your problems will be solved if you avoid using the "null" layout and learn to use the layout managers.

for instance:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class WelcomeFoo
{
  private static final Dimension BUTTON_SIZE = new Dimension(220, 35);
  private static final String TITLE_STRING = "Fubars Are Us!";
  public static final String[] BUTTON_STRINGS = 
  {
    "Fubar", "Snafu", "Subar", "Dilligafs"  
  };
  private JPanel mainPanel = new JPanel();
  

  public WelcomeFoo()
  {
    JLabel titleLabel = new JLabel(TITLE_STRING, SwingConstants.CENTER);
    titleLabel.setFont(new Font("SansSerif", Font.ITALIC | Font.BOLD, 90));
    titleLabel.setBorder(BorderFactory.createEmptyBorder(5, 5, 5, 5));
    titleLabel.setForeground(Color.white);
    
    JPanel buttonPanel = new JPanel(new GridLayout(0, 1, 0, 5));
    buttonPanel.setOpaque(false);
    for (String buttonString : BUTTON_STRINGS)
    {
      JButton button = new JButton(buttonString);
      button.setPreferredSize(BUTTON_SIZE);
      button.addActionListener(new ButtonListener());
      buttonPanel.add(button);
    }
    buttonPanel.add(new JLabel(" "));
    JButton exitBtn = new JButton("Exit");
    exitBtn.setPreferredSize(BUTTON_SIZE);
    buttonPanel.add(exitBtn);
    exitBtn.addActionListener(new ActionListener()
    {
      public void actionPerformed(ActionEvent e)
      {
        Window window = SwingUtilities.getWindowAncestor(mainPanel);
        window.dispose();
      }
    });
    JPanel btnPanelHolder = new JPanel(new FlowLayout(FlowLayout.CENTER));
    btnPanelHolder.setOpaque(false);
    btnPanelHolder.add(buttonPanel);

    int ebGap = 10;
    mainPanel.setBorder(BorderFactory.createEmptyBorder(ebGap, ebGap, ebGap, ebGap));
    mainPanel.setLayout(new BorderLayout());
    mainPanel.setBackground(Color.lightGray);
    mainPanel.add(titleLabel, BorderLayout.NORTH);
    mainPanel.add(btnPanelHolder);
  }

  public JComponent getComponent()
  {
    return mainPanel;
  }
  
  private class ButtonListener implements ActionListener
  {
    public void actionPerformed(ActionEvent e)
    {
      System.out.println("Button " + e.getActionCommand() + " was pressed");
    }
  }

  private static void createAndShowUI()
  {
    JFrame frame = new JFrame("WelcomeFoo");
    frame.getContentPane().add(new WelcomeFoo().getComponent());
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.pack();
    frame.setLocationRelativeTo(null);
    frame.setVisible(true);
  }

  public static void main(String[] args)
  {
    java.awt.EventQueue.invokeLater(new Runnable()
    {
      public void run()
      {
        createAndShowUI();
      }
    });
  }
}

[Gregory Ball]

Sorry for not having my code in quotes. first time on this website.

Cheers for the reply.

I have used layout managers before and they have worked quite well, but time consuming.

I'm still wondering whats wrong with the code. Or does Java hate doing things without a layout scheme?
Should I just forget about it and go a head with what is suggested?

Cheers Greg

[pete stein]

Gregory Ball wrote:I have used layout managers before and they have worked quite well, but time consuming.

What's more time consuming, trying to set things using absolute positioning, getting it wrong, and having to re-work the code multiple times to get it right, or using nested layouts and just letting the layout managers do the hard work for you? For me the question is a no-brainer.

I'm still wondering whats wrong with the code. Or does Java hate doing things without a layout scheme?

Java neither hates nor loves, but only does what you tell it to do. Your positionings are incorrect. Simple as that.

Should I just forget about it and go a head with what is suggested?

Your call.