GridLayout Vertical gap error

[Yohan Weerasinghe]

Hi all,

Here I am trying to adjust the Vertical gap of the GridLayout because the gap between the components are too much for me. But it seems like that code I used is not working. please help me. Here is my code.

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

public class gird extends JFrame{

    JLabel title,text;
    GridLayout layout;

    public gird()
    {
        title = new JLabel("The GridLayout",JLabel.CENTER);
        text = new JLabel("Text inside the GridLayot");

        layout = new GridLayout(2,1);


        JPanel panel = new JPanel();
        panel.setLayout(layout);
        layout.setVgap(1);
        panel.add(title);
        panel.add(text);
        panel.setBorder(new LineBorder(Color.BLACK,5));

        JPanel mPanel = (JPanel)getContentPane();
        mPanel.setLayout(new BorderLayout());
        mPanel.add(panel);

    }

    public static void main(String[]args)
    {
        gird g = new gird();
        g.setVisible(true);
        g.setSize(300,300);
        g.validate();
        g.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    }

}

[Darryl Burke]

Yohan Weerasinghe wrote:But it seems like that code I used is not working.

In what way is is 'not working'?

Some notes on your posted code:

By convention, Java class names start with an uppercase letter.

JPanel has a constructer that accepts a layout.

A JFrame's getContentPane() returns a Container. There is no need to cast to JPanel to add components.

The content pane of a JFrame has a BorderLayout by default.

There is no need to validate a GUI if all sizing etc. operations are carried out before it is made visible.

Applying these, and getting rid of unnecessary blank likes reduces your code to this:

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

public class Gird extends JFrame {

  JLabel title, text;
  GridLayout layout;

  public Gird() {
    title = new JLabel("The GridLayout", JLabel.CENTER);
    text = new JLabel("Text inside the GridLayot");

    layout = new GridLayout(2, 1);
    layout.setVgap(1);

    JPanel panel = new JPanel(layout);
    panel.setBorder(new LineBorder(Color.BLACK, 5));
    panel.add(title);
    panel.add(text);

    getContentPane().add(panel);
  }

  public static void main(String[] args) {
    Gird g = new Gird();
    g.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    g.setSize(300, 300);
    g.setVisible(true);
  }
}

[Yohan Weerasinghe]

I really appreciate those advice, they are great! Please guide me to the success like that. I am open to any advice, comment from anyone who has more knowledge than me.

In what way is is 'not working'?

The vertical gap. The vertical gap is too much. What I want is this. The first row should be lot smaller than the second row, because second row is the place I am gonna add details. I tried to adjust the gap using "setVgap(1)" but I didn't get the expected result. Please help

[Darryl Burke]

The first row should be lot smaller than the second row

Then don't use a GridLayout. Choose another layout.

[Yohan Weerasinghe]

Darryl Burke wrote:

The first row should be lot smaller than the second row

Then don't use a GridLayout. Choose another layout.

OK..Thanks.......I really appreciate your help. Marking this thread as solved.