Changeset 4110 in josm

Timestamp:

Jun 1, 2011 9:15:09 PM (2 years ago)

Author:

stoecker

Message:

see 6368 - patch by Kachkaev - Enter key pressing handler for text fields in Credentials dialog

File:

• trunk/src/org/openstreetmap/josm/gui/io/CredentialDialog.java (modified) (9 diffs)

trunk/src/org/openstreetmap/josm/gui/io/CredentialDialog.java

@@ -15 +15 @@
 import java.awt.event.FocusEvent;
 import java.awt.event.KeyEvent;
+import java.awt.event.KeyListener;
 import java.awt.event.WindowAdapter;
 import java.awt.event.WindowEvent;
@@ -104 +105 @@
     public void prepareForOsmApiCredentials(String username, String password) {
         setTitle(tr("Enter credentials for OSM API"));
-        getContentPane().add(pnlCredentials = new OsmApiCredentialsPanel(), BorderLayout.CENTER);
+        getContentPane().add(pnlCredentials = new OsmApiCredentialsPanel(this), BorderLayout.CENTER);
         pnlCredentials.init(username, password);
         validate();
@@ -111 +112 @@
     public void prepareForProxyCredentials(String username, String password) {
         setTitle(tr("Enter credentials for HTTP proxy"));
-        getContentPane().add(pnlCredentials = new HttpProxyCredentialsPanel(), BorderLayout.CENTER);
+        getContentPane().add(pnlCredentials = new HttpProxyCredentialsPanel(this), BorderLayout.CENTER);
         pnlCredentials.init(username, password);
         validate();
@@ -137 +138 @@
         protected JMultilineLabel lblHeading;
         protected JMultilineLabel lblWarning;
+        protected CredentialDialog owner; // owner Dependency Injection to use Key listeners for username and password text fields
 
         protected void build() {
@@ -143 +145 @@
             tfUserName.addFocusListener(new SelectAllOnFocusHandler());
             tfPassword.addFocusListener(new SelectAllOnFocusHandler());
+            tfUserName.addKeyListener(new TFKeyListener(owner, tfUserName, tfPassword));
+            tfPassword.addKeyListener(new TFKeyListener(owner, tfPassword, tfUserName));
             cbSaveCredentials =  new JCheckBox(tr("Save user and password (unencrypted)"));
 
@@ -202 +206 @@
         }
 
-        public CredentialPanel() {
+        public CredentialPanel(CredentialDialog owner) {
+            this.owner = owner;
         }
 
@@ -243 +248 @@
         }
 
-        public OsmApiCredentialsPanel() {
+        public OsmApiCredentialsPanel(CredentialDialog owner) {
+            super(owner);
             build();
         }
@@ -260 +266 @@
         }
 
-        public HttpProxyCredentialsPanel() {
+        public HttpProxyCredentialsPanel(CredentialDialog owner) {
+            super(owner);
             build();
         }
@@ -275 +282 @@
     }
 
+    /**
+     * Listener for username and password text fields key events.
+     * When user presses Enter:
+     *   If current text field is empty (or just contains a sequence of spaces), nothing happens (or all spaces become selected).
+     *   If current text field is not empty, but the next one is (or just contains a sequence of spaces), focuses the next text field.
+     *   If both text fields contain characters, submits the form by calling owner's {@link OKAction}.
+     */
+    static private class TFKeyListener implements KeyListener{
+        protected CredentialDialog owner; // owner Dependency Injection to call OKAction
+        protected JTextField currentTF;
+        protected JTextField nextTF;
+
+        public TFKeyListener (CredentialDialog owner, JTextField currentTF, JTextField nextTF)
+        {
+            this.owner = owner;
+            this.currentTF = currentTF;
+            this.nextTF = nextTF;
+        }
+
+        public void keyPressed(KeyEvent e) {
+            if(e.getKeyChar() == KeyEvent.VK_ENTER) {
+                if (currentTF.getText().trim().isEmpty()) {
+                    currentTF.selectAll();
+                    return;
+                } else if (nextTF.getText().trim().isEmpty()) {
+                    nextTF.requestFocusInWindow();
+                    nextTF.selectAll();
+                    return;
+                } else {
+                    OKAction okAction = owner.new OKAction();
+                    okAction.actionPerformed(null);
+                }
+            }
+        }
+
+        public void keyReleased ( KeyEvent e ){
+        }
+
+        public void keyTyped ( KeyEvent e ){
+        }
+    }
+
     class OKAction extends AbstractAction {
         public OKAction() {