JAL-3633 Small changes to allow future fitting with ApplicationSingletonI/Jalview...

[jalview.git] / src / jalview / jbgui / GPreferences.java

diff --git a/src/jalview/jbgui/GPreferences.java b/src/jalview/jbgui/GPreferences.java
index f82c798..624041c 100755 (executable)
--- a/src/jalview/jbgui/GPreferences.java
+++ b/src/jalview/jbgui/GPreferences.java
@@ -69,6 +69,8 @@ import javax.swing.border.EtchedBorder;
 import javax.swing.border.TitledBorder;
 import javax.swing.event.ChangeEvent;
 import javax.swing.event.ChangeListener;
+import javax.swing.event.DocumentEvent;
+import javax.swing.event.DocumentListener;
 import javax.swing.table.TableCellEditor;
 import javax.swing.table.TableCellRenderer;
 
@@ -259,7 +261,7 @@ public class GPreferences extends JPanel
 
   protected JTextField proxyAuthUsernameTB = new JTextField();
 
-  protected JPasswordField proxyAuthPasswordTB = new JPasswordField();
+  protected JPasswordField proxyAuthPasswordPB = new JPasswordField();
 
   protected JTextField defaultBrowser = new JTextField();
 
@@ -271,6 +273,8 @@ public class GPreferences extends JPanel
 
   protected JRadioButton customProxy = new JRadioButton();
 
+  protected JButton applyProxyButton = new JButton();
+
   protected JCheckBox usagestats = new JCheckBox();
 
   protected JCheckBox questionnaire = new JCheckBox();
@@ -366,24 +370,18 @@ public class GPreferences extends JPanel
 
   protected JTextArea backupfilesExampleLabel = new JTextArea();
 
+  private final JTabbedPane tabbedPane = new JTabbedPane();
+
+  private JLabel messageLabel = new JLabel("", JLabel.CENTER);
+
   /**
    * Creates a new GPreferences object.
    */
   public GPreferences()
   {
-    new GPreferences(0);
-  }
-
-  public GPreferences(int selectTab)
-  {
-    new GPreferences(selectTab, null);
-  }
-
-  public GPreferences(int selectTab, String message)
-  {
     try
     {
-      jbInit(selectTab, message);
+      jbInit();
     } catch (Exception ex)
     {
       ex.printStackTrace();
@@ -397,28 +395,11 @@ public class GPreferences extends JPanel
    */
   private void jbInit() throws Exception
   {
-    jbInit(0);
-  }
-
-  private void jbInit(int selectTab) throws Exception
-  {
-    jbInit(selectTab, null);
-  }
-
-  public final static int CONNECTIONS_TAB = 5;
-
-  private void jbInit(int selectTab, String message) throws Exception
-  {
-    final JTabbedPane tabbedPane = new JTabbedPane();
+    // final JTabbedPane tabbedPane = new JTabbedPane();
     this.setLayout(new BorderLayout());
 
-    if (message != null)
-    {
-      JLabel messageLabel = new JLabel(message, JLabel.CENTER);
-      messageLabel.setFont(LABEL_FONT_BOLD);
-      messageLabel.setForeground(Color.RED.darker());
-      this.add(messageLabel, BorderLayout.NORTH);
-    }
+    // message label at top
+    this.add(messageLabel, BorderLayout.NORTH);
 
     JPanel okCancelPanel = initOkCancelPanel();
     this.add(tabbedPane, BorderLayout.CENTER);
@@ -460,6 +441,7 @@ public class GPreferences extends JPanel
     /*
      * Handler to validate a tab before leaving it - currently only for
      * Structure.
+     * Adding a clearMessage() so messages are cleared when changing tabs.
      */
     tabbedPane.addChangeListener(new ChangeListener()
     {
@@ -478,10 +460,47 @@ public class GPreferences extends JPanel
           }
         }
         lastTab = tabbedPane.getSelectedComponent();
+
+        clearMessage();
       }
 
     });
+  }
+
+  public void setMessage(String message)
+  {
+    if (message != null)
+    {
+      messageLabel.setText(message);
+      messageLabel.setFont(LABEL_FONT_BOLD);
+      messageLabel.setForeground(Color.RED.darker());
+      messageLabel.revalidate();
+      messageLabel.repaint();
+    }
+    // note message not cleared if message is null. call clearMessage()
+    // directly.
+    this.revalidate();
+    this.repaint();
+  }
 
+  public void clearMessage()
+  {
+    // only repaint if message exists
+    if (messageLabel.getText() != null
+            && messageLabel.getText().length() > 0)
+    {
+      messageLabel.setText("");
+      messageLabel.revalidate();
+      messageLabel.repaint();
+      this.revalidate();
+      this.repaint();
+    }
+  }
+
+  public final static int CONNECTIONS_TAB = 5;
+
+  public void selectTab(int selectTab)
+  {
     // select a given tab - currently only for Connections
     switch (selectTab)
     {
@@ -490,7 +509,6 @@ public class GPreferences extends JPanel
       break;
     default:
     }
-
   }
 
   /**
@@ -932,8 +950,30 @@ public class GPreferences extends JPanel
     proxyPortHttpsTB.setColumns(4);
     proxyAuthUsernameTB.setFont(LABEL_FONT);
     proxyAuthUsernameTB.setColumns(30);
-    proxyAuthPasswordTB.setFont(LABEL_FONT);
-    proxyAuthPasswordTB.setColumns(30);
+    proxyAuthPasswordPB.setFont(LABEL_FONT);
+    proxyAuthPasswordPB.setColumns(30);
+    proxyAuthPasswordPB.getDocument()
+            .addDocumentListener(new DocumentListener()
+            {
+              @Override
+              public void changedUpdate(DocumentEvent e)
+              {
+                proxyAuthPasswordHighlight(true);
+              }
+
+              @Override
+              public void insertUpdate(DocumentEvent e)
+              {
+                proxyAuthPasswordHighlight(true);
+              }
+
+              @Override
+              public void removeUpdate(DocumentEvent e)
+              {
+                proxyAuthPasswordHighlight(true);
+              }
+
+            });
 
     // Label for Port text box
     portLabel.setFont(LABEL_FONT);
@@ -1125,7 +1165,7 @@ public class GPreferences extends JPanel
     c.gridx++;
     c.weightx = 1.0;
     c.anchor = GridBagConstraints.LINE_START;
-    upPanel.add(proxyAuthPasswordTB, c);
+    upPanel.add(proxyAuthPasswordPB, c);
 
     c.gridx++;
     c.weightx = 0.4;
@@ -1135,9 +1175,45 @@ public class GPreferences extends JPanel
     gbc.gridy++;
     proxyPanel.add(upPanel, gbc);
 
+    applyProxyButton.setText(MessageManager.getString("action.apply"));
+    applyProxyButton.addActionListener(new ActionListener()
+    {
+      @Override
+      public void actionPerformed(ActionEvent e)
+      {
+        saveProxySettings();
+      }
+    });
+    gbc.gridy++;
+    gbc.fill = GridBagConstraints.NONE;
+    gbc.anchor = GridBagConstraints.LINE_END;
+    proxyPanel.add(applyProxyButton, gbc);
+
     return proxyPanel;
   }
 
+  public void proxyAuthPasswordHighlight(boolean enabled)
+  {
+    if (enabled && proxyType.isSelected(customProxy.getModel())
+            && proxyAuth.isSelected()
+            && !proxyAuthUsernameTB.getText().isEmpty()
+            && proxyAuthPasswordPB.getDocument().getLength() == 0)
+    {
+
+      proxyAuthPasswordPB.grabFocus();
+      proxyAuthPasswordPB.setBackground(Color.PINK);
+    }
+    else
+    {
+      proxyAuthPasswordPB.setBackground(Color.WHITE);
+    }
+  }
+
+  public void saveProxySettings()
+  {
+    // overridden in Preferences
+  }
+
   private String displayUserHostPort(String user, String host, String port)
   {
     boolean hostBlank = (host == null || host.isEmpty());
@@ -3025,7 +3101,7 @@ public class GPreferences extends JPanel
     proxyAuthPasswordLabel.setEnabled(enabled);
     passwordNotStoredLabel.setEnabled(enabled);
     proxyAuthUsernameTB.setEnabled(enabled);
-    proxyAuthPasswordTB.setEnabled(enabled);
+    proxyAuthPasswordPB.setEnabled(enabled);
   }
 
   public void setCustomProxyEnabled()
@@ -3048,11 +3124,13 @@ public class GPreferences extends JPanel
   public void proxyType_actionPerformed()
   {
     setCustomProxyEnabled();
+    proxyAuthPasswordHighlight(true);
   }
 
   public void proxyAuth_actionPerformed()
   {
     setProxyAuthEnabled();
+    proxyAuthPasswordHighlight(true);
   }
 
   /**