Name: jk109818 Date: 09/08/2003


FULL PRODUCT VERSION :
java version "1.4.1_02"
Java(TM) 2 Runtime Environment, Standard Edition (build 1.4.1_02-b06)
Java HotSpot(TM) Client VM (build 1.4.1_02-b06, mixed mode)

FULL OS VERSION :
Microsoft Windows 2000 [Version 5.00.2195]

A DESCRIPTION OF THE PROBLEM :
The JDialog does not layout properly after a call to setBounds (or any of its cousins i.e. setSize, etc.). After you perform the steps to see the layout problem the dialog will be redrawn correctly after a manual resize of the dialog.

STEPS TO FOLLOW TO REPRODUCE THE PROBLEM :
Run the source code given. Press the button. You should see that the button is not redrawn or layed out properly. I tried to call the invalidate, repaint, and doLayout methods all of which do not result in any change.

EXPECTED VERSUS ACTUAL BEHAVIOR :
EXPECTED -
Expected behavior is the dialog redrawn correctly to the new bounds.
ACTUAL -
Incorrectly layed out components in the dialog conforming to the old size of the dialog.

REPRODUCIBILITY :
This bug can be reproduced always.

---------- BEGIN SOURCE ----------
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.*;

public class Bug
{
    public static void main(String[] args)
    {
        class MaxListener implements ActionListener
        {
            JDialog dialog;
            MaxListener(JDialog dialog)
            {
                this.dialog = dialog;
            }
            public void actionPerformed(ActionEvent e)
            {
                Dimension dim = dialog.getToolkit().getScreenSize();
                dialog.setBounds(0, 0, dim.width - 100, dim.height - 100);
                // uncomment to see workaround
                // dialog.setSize(dim.width, dim.height);
            }
        }
        JFrame frame = new JFrame();
        JDialog dialog = new JDialog(frame, true);
        JButton button = new JButton("Maximize");
        button.addActionListener(new MaxListener(dialog));
        dialog.getContentPane().add(button);
        dialog.setLocationRelativeTo(null);
        dialog.setSize(100, 100);

        dialog.show();
        System.exit(0);
    }
}

---------- END SOURCE ----------

CUSTOMER SUBMITTED WORKAROUND :
call setBounds twice with different sizes
ifI want a bigger dialog I can just call the first one slightly smaller and then the second one with the correct size
(Incident Review ID: 186636)
======================================================================

Name: jk109818 Date: 09/08/2003


FULL PRODUCT VERSION :
On Windows:
java version "1.4.1_02"
Java(TM) 2 Runtime Environment, Standard Edition (build 1.4.1_02-b06)
Java HotSpot(TM) Client VM (build 1.4.1_02-b06, mixed mode)

On Linux:
Java version "1.4.1_02"
Java(TM) 2 Runtime Environment, Standard Edition (build 1.4.1_02-b06)
Java HotSpot(TM) Client VM (build 1.4.1_02-b06, mixed mode)

FULL OS VERSION :
Windows XP, Linux 2.4.20

A DESCRIPTION OF THE PROBLEM :
In a JFrame with several internal components, if, after the components are added to the frame, setVisible(true) is called and then setBounds(0,0,600,450), the components within the frame are not repainted. (On Linux, the frame does not change size. On Windows, the frame changes size, but none of the components appear in the frame.)

I ran into the problem in an example from the book Swing, Second Edition by Matthew Robinson and Pavel Vorobiev from Manning Publishing. Chapter 2, example 5: FocusDemo. The code for this example is available from http://www.manning.com/robinson2. I've also pasted a copy of the code below. (Copyright goes to Robinson and Vorobiev) The example as coded in the book fails as described. If the setVisible() is moved one line down (below the setBounds()) in main(), then the frame displays properly. If the corner of the frame is dragged slightly (in or out; doesn't matter) then it is displayed properly.

Changing the size of the frame externally (dragging) causes the frame to be invalidated, and the frame is redrawn. But changing the frame within the program (using setBounds()) doesn't seem to have the same effect.

The problem can also be demonstrated by placing a setBounds(0,0,200,200) before the setVisible in main. When the window is displayed, on Linux it will stay at 200 by 200, and on Windows it will display as 600 by 450, but only the 200 by 200 area will be painted.

Robinson and Vorobiev's code, w/ my changes to better demonstrate the problem is attached below.


STEPS TO FOLLOW TO REPRODUCE THE PROBLEM :
Run the code above. On my Windows XP system, it will create a 600 by 450 window with only 200 by 200 painted. Dragging a corner slightly will cause the entire window to be painted correctly. On my Linux system, it will create a 200 by 200 window, ignoring the final call to setBounds().

EXPECTED VERSUS ACTUAL BEHAVIOR :
EXPECTED -
I would expect the call to setBounds to invalidate the frame and cause it to be repainted if it was already visible.
ACTUAL -
The call to setBounds() is honored on one system but not the other, and neither system bothers to repaint the frame within the new bounds.

ERROR MESSAGES/STACK TRACES THAT OCCUR :
No error messages are produced.

REPRODUCIBILITY :
This bug can be reproduced always.

---------- BEGIN SOURCE ----------
/**
 * Copyright 1999-2002 Matthew Robinson and Pavel Vorobiev.
 * All Rights Reserved.
 *
 * ===================================================
 * This program contains code from the book "Swing"
 * 2nd Edition by Matthew Robinson and Pavel Vorobiev
 * http://www.spindoczine.com/sbe
 * ===================================================
 *
 * The above paragraph must be included in full, unmodified
 * and completely intact in the beginning of any source code
 * file that references, copies or uses (in any way, shape
 * or form) code contained in this file.
 */

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

public class FocusDemo extends JFrame {

  public FocusDemo() {
    super("Focus Demo");
    
    System.out.println("Creating object");

    JPanel contentPane = (JPanel) getContentPane();
    contentPane.setBorder(new TitledBorder("Focus Cycle A"));
    contentPane.add(createComponentPanel(), BorderLayout.NORTH);
    JDesktopPane desktop1 = new JDesktopPane();
    contentPane.add(desktop1, BorderLayout.CENTER);
    
    System.out.println("Cycle A created");

    JInternalFrame internalFrame1 = new JInternalFrame("Focus Cycle B", true, true, true, true);
    contentPane = (JPanel) internalFrame1.getContentPane();
    contentPane.add(createComponentPanel(), BorderLayout.NORTH);
    JDesktopPane desktop2 = new JDesktopPane();
    contentPane.add(desktop2, BorderLayout.CENTER);
    desktop1.add(internalFrame1);
    internalFrame1.setBounds(20,20,500,300);
    internalFrame1.show();
    
    System.out.println("Cycle B created");

    JInternalFrame internalFrame2 = new JInternalFrame("Focus Cycle C", true, true, true, true);
    contentPane = (JPanel) internalFrame2.getContentPane();
    contentPane.add(createComponentPanel(), BorderLayout.NORTH);
    JDesktopPane desktop3 = new JDesktopPane();
    contentPane.add(desktop3, BorderLayout.CENTER);
    desktop2.add(internalFrame2);
    internalFrame2.setBounds(20,20,400,200);
    internalFrame2.show();

    JInternalFrame internalFrame3 = new JInternalFrame("Focus Cycle D", false, true, true, true);
    contentPane = (JPanel) internalFrame3.getContentPane();
    contentPane.add(createComponentPanel(), BorderLayout.NORTH);
    desktop3.add(internalFrame3);
    internalFrame3.setBounds(20,20,300,100);
    internalFrame3.show();
    
    System.out.println("finished object");
  }

  public static void main(String[] args) {
      System.out.println("new FocusDemo");
    FocusDemo frame = new FocusDemo();
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); // 1.3

    /* Original final code:
     * frame.setVisible(true);
     * frame.setBounds(0,0,600,450);
     */
    
    /* Ending by rpnix, that better demonstrates the problem w/ setBounds()
     * and repainting the frame: */
    frame.setBounds(0,0,200,200);
    frame.setVisible(true);
    frame.setBounds(0,0,600,450);
  }

  protected JPanel createComponentPanel() {
      System.out.println("Creating panel");
    JPanel panel = new JPanel();
    panel.add(new JButton("Button 1"));
    panel.add(new JButton("Button 2"));
    panel.add(new JTextField(10));
    return panel;
  }
}

---------- END SOURCE ----------

CUSTOMER SUBMITTED WORKAROUND :
Workaround is to call setVisible() after the setBounds(). But this would imply that you would have to call setVisible() after any setBounds() change. This is not true of other changes to components; otherwise, our code would be completely littered with setVisible() calls.
(Review ID: 186755)
======================================================================