Problem with Combo Box in a Dialog Box

Question

I have a dialog box that includes a combo box.
For some reason the combo box shows up under the first text box. In other words, the combo box is not separate from the first field. It has the following:
ComboBox with Sequence showing, then Enter Identifyer
Textbox URL
Textbox Enter Resource 1
Textbox Enter Resource 2
Textbox Enter Resource 3
Textbox Enter Resource 4
Textbox Enter Resource 5
It's putting the combo box in the first text field...it should be
ComboBox with Sequence showing
URL Textbox
Enter Identifyer Textbox
Enter Resource 1 Textbox
Enter Resource 2 Textbox...
Here is my code:
import java.awt.*;
import java.awt.event.*;
import java.io.*;
import java.util.*;
import javax.swing.*;
import javax.swing.border.*;
import javax.swing.event.*;
import dl.*;
* Dialog to enter container information
public class AddContainer extends JDialog {
private JTextField valueBox1;
private JTextField valueBox2;
private JTextField valueBox3;
private JTextField valueBox4;
private JTextField valueBox5;
public String value1;
public String value2;
public String value3;
public String value4;
public String value5;
private JTextField identifyerBox;
private JTextField URLBox;
private JTextField attrBox;
public String identifyer;
public String URL;
public int choice;
* Constructor.
public AddContainer(Frame parent) {
super(parent, "Add Container", true);
JPanel pp = new JPanel(new DialogLayout2());
pp.setBorder(new CompoundBorder(
new EtchedBorder(EtchedBorder.RAISED),
new EmptyBorder(5,5,5,5)));
String[] ContStrings = {  "Sequence", "Bag", "Alternative" };
// Add action listener.
ActionListener contlst = new ActionListener() {
public void actionPerformed(ActionEvent e) {
JComboBox cb = (JComboBox)e.getSource();
int choice = (int)cb.getSelectedIndex();
//Create the combo box, select the item at index 0.
//Indices start at 0, so 2 specifies the Alternative
JComboBox ContList = new JComboBox(ContStrings);
ContList.setSelectedIndex(0);
ContList.addActionListener(contlst);
//Add combo box to panel.
pp.add(ContList);
pp.add(new JLabel("Enter Identifyer"));
identifyerBox = new JTextField(16);
pp.add(identifyerBox);
pp.add(new JLabel("URL"));
URLBox = new JTextField(16);
pp.add(URLBox);
pp.add(new JLabel("Enter Resource 1"));
valueBox1 = new JTextField(25);
pp.add(valueBox1);
pp.add(new JLabel("Enter Resource 2"));
valueBox2 = new JTextField(25);
pp.add(valueBox2);
pp.add(new JLabel("Enter Resource 3"));
valueBox3 = new JTextField(25);
pp.add(valueBox3);
pp.add(new JLabel("Enter Resource 4"));
valueBox4 = new JTextField(25);
pp.add(valueBox4);
pp.add(new JLabel("Enter Resource 5"));
valueBox5 = new JTextField(25);
pp.add(valueBox5);
JPanel p = new JPanel(new DialogLayout2());
p.setBorder(new EmptyBorder(10, 10, 10, 10));
p.add(pp);
ActionListener lst = new ActionListener() {
public void actionPerformed(ActionEvent evt) {
identifyer = identifyerBox.getText();
URL = URLBox.getText();
value1 = valueBox1.getText();
value2 = valueBox2.getText();
value3 = valueBox3.getText();
value4 = valueBox4.getText();
value5 = valueBox5.getText();
dispose();
JButton saveButton = new JButton("ADD");
saveButton.addActionListener(lst);
getRootPane().setDefaultButton(saveButton);
getRootPane().registerKeyboardAction(lst,
KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, 0),
JComponent.WHEN_IN_FOCUSED_WINDOW);
p.add(saveButton);
JButton cancelButton = new JButton("Cancel");
lst = new ActionListener() {
public void actionPerformed(ActionEvent evt) {
dispose();
cancelButton.addActionListener(lst);
getRootPane().registerKeyboardAction(lst,
KeyStroke.getKeyStroke(KeyEvent.VK_ESCAPE, 0),
JComponent.WHEN_IN_FOCUSED_WINDOW);
p.add(cancelButton);
getContentPane().add(p, BorderLayout.CENTER);
pack();
setResizable(false);
setLocationRelativeTo(parent);

Answer 1

Seems the problem is in your DialogLayout2 class which must be in package dl. I tried a grid layout and got something that looks like what you described. I laid out pp with a gridbag layout.
import java.awt.*;
import java.awt.event.*;
import java.io.*;
import java.util.*;
import javax.swing.*;
import javax.swing.border.*;
import javax.swing.event.*;
//import dl.*;
* Dialog to enter container information
public class jello extends JDialog {
  private JTextField valueBox1;
  private JTextField valueBox2;
  private JTextField valueBox3;
  private JTextField valueBox4;
  private JTextField valueBox5;
  public String value1;
  public String value2;
  public String value3;
  public String value4;
  public String value5;
  private JTextField identifyerBox;
  private JTextField URLBox;
  private JTextField attrBox;
  public String identifyer;
  public String URL;
  public int choice;
  * Constructor.
  public jello(JFrame parent) {
    super(parent, "Add Container", true);
    GridBagLayout gridbag = new GridBagLayout();
    GridBagConstraints gbc = new GridBagConstraints();
    JPanel pp = new JPanel(gridbag);
                        //(new GridLayout(0,2));
                        //(new DialogLayout2());
    pp.setBackground(Color.red);
    pp.setBorder(
      new CompoundBorder(
        new EtchedBorder(EtchedBorder.RAISED),
        new EmptyBorder(5,5,5,5)));
    String[] ContStrings = { "Sequence", "Bag", "Alternative" };
    // Add action listener.
    ActionListener contlst = new ActionListener() {
      public void actionPerformed(ActionEvent e) {
        JComboBox cb = (JComboBox)e.getSource();
        int choice = (int)cb.getSelectedIndex();
    //Create the combo box, select the item at index 0.
    //Indices start at 0, so 2 specifies the Alternative
    JComboBox ContList = new JComboBox(ContStrings);
    ContList.setSelectedIndex(0);
    ContList.addActionListener(contlst);
    //Add combo box to panel.
    gbc.insets = new Insets(2,2,2,2);
    gbc.anchor = gbc.WEST;
    gbc.gridwidth = gbc.REMAINDER;
    pp.add(ContList, gbc);
    gbc.anchor = gbc.EAST;
    gbc.gridwidth = gbc.RELATIVE;
    pp.add(new JLabel("Enter Identifyer"), gbc);
    identifyerBox = new JTextField(16);
    gbc.anchor = gbc.WEST;
    gbc.gridwidth = gbc.REMAINDER;
    pp.add(identifyerBox, gbc);
    gbc.anchor = gbc.EAST;
    gbc.gridwidth = gbc.RELATIVE;
    pp.add(new JLabel("URL"), gbc);
    URLBox = new JTextField(16);
    gbc.anchor = gbc.WEST;
    gbc.gridwidth = gbc.REMAINDER;
    pp.add(URLBox, gbc);
    gbc.anchor = gbc.CENTER;
    gbc.gridwidth = gbc.RELATIVE;
    pp.add(new JLabel("Enter Resource 1"), gbc);
    valueBox1 = new JTextField(25);
    gbc.gridwidth = gbc.REMAINDER;
    pp.add(valueBox1, gbc);
    gbc.gridwidth = gbc.RELATIVE;
    pp.add(new JLabel("Enter Resource 2"), gbc);
    valueBox2 = new JTextField(25);
    gbc.gridwidth = gbc.REMAINDER;
    pp.add(valueBox2, gbc);
    gbc.gridwidth = gbc.RELATIVE;
    pp.add(new JLabel("Enter Resource 3"), gbc);
    valueBox3 = new JTextField(25);
    gbc.gridwidth = gbc.REMAINDER;
    pp.add(valueBox3, gbc);
    gbc.gridwidth = gbc.RELATIVE;
    pp.add(new JLabel("Enter Resource 4"), gbc);
    valueBox4 = new JTextField(25);
    gbc.gridwidth = gbc.REMAINDER;
    pp.add(valueBox4, gbc);
    gbc.gridwidth = gbc.RELATIVE;
    pp.add(new JLabel("Enter Resource 5"), gbc);
    valueBox5 = new JTextField(25);
    gbc.gridwidth = gbc.REMAINDER;
    pp.add(valueBox5, gbc);
    JPanel p = new JPanel();//(new DialogLayout2());
    p.setBackground(Color.blue);
    p.setBorder(new EmptyBorder(10, 10, 10, 10));
    p.add(pp);
    ActionListener lst = new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        identifyer = identifyerBox.getText();
        URL = URLBox.getText();
        value1 = valueBox1.getText();
        value2 = valueBox2.getText();
        value3 = valueBox3.getText();
        value4 = valueBox4.getText();
        value5 = valueBox5.getText();
        dispose();
    JButton saveButton = new JButton("ADD");
    saveButton.addActionListener(lst);
    getRootPane().setDefaultButton(saveButton);
    getRootPane().registerKeyboardAction(lst,
      KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, 0),
      JComponent.WHEN_IN_FOCUSED_WINDOW);
    p.add(saveButton);
    JButton cancelButton = new JButton("Cancel");
    lst = new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        dispose();
    cancelButton.addActionListener(lst);
    getRootPane().registerKeyboardAction(lst,
      KeyStroke.getKeyStroke(KeyEvent.VK_ESCAPE, 0),
      JComponent.WHEN_IN_FOCUSED_WINDOW);
    p.add(cancelButton);
    parent.getContentPane().add(p, BorderLayout.CENTER);
    parent.pack();
    parent.setResizable(false);
//    setLocationRelativeTo(parent);
    parent.setVisible(true);
  public static void main(String[] args) {
    JFrame frame = new JFrame();
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    new jello(frame);
    frame.setLocation(0,200);
}