X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FEditNameDialog.java;h=6d5cb4662e15287d4da38b3e2f863daa464f3101;hb=1ef93cb2817f75699773f049b8d25e5b509c2830;hp=0ec4a879be21fecf28c77e1b29ca55d019704874;hpb=fee1b781ca14aadea5d112fc554fe14879c787c5;p=jalview.git

diff --git a/src/jalview/gui/EditNameDialog.java b/src/jalview/gui/EditNameDialog.java
index 0ec4a87..6d5cb46 100644
--- a/src/jalview/gui/EditNameDialog.java
+++ b/src/jalview/gui/EditNameDialog.java
@@ -20,27 +20,35 @@
  */
 package jalview.gui;
 
-import jalview.util.MessageManager;
-
-import java.awt.BorderLayout;
+import java.awt.FlowLayout;
 import java.awt.Font;
 
+import javax.swing.BoxLayout;
 import javax.swing.JButton;
 import javax.swing.JComponent;
 import javax.swing.JLabel;
-import javax.swing.JOptionPane;
 import javax.swing.JPanel;
 import javax.swing.JTextField;
 
+import jalview.util.MessageManager;
+
+/**
+ * A dialog where a name and description may be edited
+ */
 public class EditNameDialog
 {
-  JTextField id, description;
+  private static final Font COURIER_12 = new Font("Courier", Font.PLAIN,
+          12);
+
+  JTextField id;
+
+  JTextField description;
 
   JButton ok = new JButton(MessageManager.getString("action.accept"));
 
   JButton cancel = new JButton(MessageManager.getString("action.cancel"));
 
-  boolean accept = false;
+  private JPanel panel;
 
   public String getName()
   {
@@ -59,36 +67,68 @@ public class EditNameDialog
     }
   }
 
+  /**
+   * Constructor
+   * 
+   * @param name
+   * @param desc
+   * @param label1
+   * @param label2
+   */
   public EditNameDialog(String name, String desc, String label1,
-          String label2, String title, JComponent parent)
+          String label2)
   {
-    JLabel idlabel = new JLabel(label1);
-    JLabel desclabel = new JLabel(label2);
-    idlabel.setFont(new Font("Courier", Font.PLAIN, 12));
-    desclabel.setFont(new Font("Courier", Font.PLAIN, 12));
+    panel = new JPanel();
+    panel.setLayout(new BoxLayout(panel, BoxLayout.Y_AXIS));
+    JPanel namePanel = new JPanel(new FlowLayout(FlowLayout.RIGHT));
+    JPanel descriptionPanel = new JPanel(new FlowLayout(FlowLayout.RIGHT));
+    panel.add(namePanel);
+    panel.add(descriptionPanel);
+
+    JLabel nameLabel = new JLabel(label1);
+    // nameLabel.setFont(COURIER_12);
+    namePanel.add(nameLabel);
+
     id = new JTextField(name, 40);
+    namePanel.add(id);
+
     description = new JTextField(desc, 40);
-    JPanel panel = new JPanel(new BorderLayout());
-    JPanel panel2 = new JPanel(new BorderLayout());
-    panel2.add(idlabel, BorderLayout.WEST);
-    panel2.add(id, BorderLayout.CENTER);
-    panel.add(panel2, BorderLayout.NORTH);
+
+    /*
+     * optionally add field for description
+     */
     if (desc != null || label2 != null)
     {
-      panel2 = new JPanel(new BorderLayout());
-      panel2.add(desclabel, BorderLayout.WEST);
-      panel2.add(description, BorderLayout.CENTER);
-      panel.add(panel2, BorderLayout.SOUTH);
-    }
-    int reply = JvOptionPane.showInternalConfirmDialog(parent, panel, title,
-            JvOptionPane.OK_CANCEL_OPTION);
-    if (!parent.requestFocusInWindow())
-    {
-      System.err.println("Bad focus for dialog!");
-    }
-    if (reply == JvOptionPane.OK_OPTION)
-    {
-      accept = true;
+      JLabel descLabel = new JLabel(label2);
+      // descLabel.setFont(COURIER_12);
+      descriptionPanel.add(descLabel);
+      descriptionPanel.add(description);
     }
   }
+
+  /**
+   * Shows the dialog, and runs the response action if OK is selected
+   * 
+   * @param action
+   */
+  public void showDialog(JComponent parent, String title, Runnable action)
+  {
+    String ok = MessageManager.getString("action.ok");
+    String cancel = MessageManager.getString("action.cancel");
+    String[] options = new String[] { ok, cancel };
+
+    JvOptionPane.newOptionDialog(parent)
+            .setResponseHandler(JvOptionPane.OK_OPTION, action)
+            .showInternalDialog(panel, title, JvOptionPane.OK_CANCEL_OPTION,
+                    JvOptionPane.PLAIN_MESSAGE, null, options, ok);
+
+    /*
+    List<Runnable> actions = new ArrayList<>();
+    actions.add(action);
+    actions.add(JvOptionPane.NULLCALLABLE);
+    
+    JvOptionPane.frameDialog(panel, title, JvOptionPane.PLAIN_MESSAGE,
+            options, ok, actions, false);
+    */
+  }
 }