Window Size is smaller than it should be

Question

Alright so I've got this JFrame with a screen on it. I've set the size to 800 by 800. However the window is created smaller than that. It's not a problem with the taskbar because it's not fullsize.

package sharph;

import java.awt.Dimension;
import javax.swing.JFrame;

public class Main extends JFrame {

    public static String Title = "Game 1";

    public static Dimension screenSize = new Dimension(800,800);

    public static void main(String[] args) {        
        JFrame frame = new JFrame();

        frame.setTitle(Title);
        frame.setSize(screenSize);
        frame.setLocationRelativeTo(null);
        frame.setResizable(true);
        frame.setDefaultCloseOperation(EXIT_ON_CLOSE);

        Screen screen = new Screen();
        screen.setSize(screenSize);

        frame.add(screen);
        frame.setVisible(true);
    }
}

In the screen class the paint method draws a box around where the border should be:

//Draw border
g.setColor(Color.RED);
g.drawRect(1, 1, 799, 799);

When I run it, the window is smaller than the box and the bottom and right sides are cut off.

enter image description here

Note the second picture I manually re-sized to show the border difference.

I realize that I have drawn the box 1 pixel smaller on each side, but the difference is much more than 2 pixels.

800*800 is the size with the window decoration (the frame and the title bar) not the size of the inner space. — MrSmith42, Apr 19 '13 at 23:32

score 6 · Accepted Answer · edited May 23 '17 at 12:13

6

This happens because the content needs to squeezed into the size of the frame minus its borders.

Checkout this question and this question for a more detailed explanation

The layout manager is also overriding the size property you set on the Screen component. In either case, you should be overriding the getPreferredSize method of the Screen class

Also, you shouldn't be relying on magic numbers or assumptions about the actual size of the component, but should, instead, be using getWidth and getHeight instead (I know, it's just for demonstration purposes)

edited May 23 '17 at 12:13

Community

1
1

answered Apr 19 '13 at 23:33

MadProgrammer

343,457
22
230
366

Thanks so much! And yes I'll change those. – Sharph Apr 20 '13 at 01:00

score 1 · Answer 2 · answered Apr 20 '13 at 00:53

1

Instead of "screen.setSize(screenSize);" type "screen.setPreferredSize(screenSize);" and then after you type "frame.setVisible(true);" type "frame.pack()". You can also remove "frame.setSize(screenSize);" if you want to.

answered Apr 20 '13 at 00:53

Michael Hall

187
5

You should, generally, pack the frame before making it visible, my cause repaint issues otherwise – MadProgrammer Apr 20 '13 at 01:14
Ah, thanks. I've never had any issues with that, but I suppose others could. – Michael Hall Apr 20 '13 at 01:16

Window Size is smaller than it should be

2 Answers2

Linked