Changeset 7545 in josm

Timestamp:

2014-09-16T19:15:00+02:00 (10 years ago)

Author:

Don-vip

Message:

fix issue with ExtendedDialog where default button cannot be activated with Enter if focus is on a multiline label

File:

• trunk/src/org/openstreetmap/josm/gui/ExtendedDialog.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/gui/ExtendedDialog.java

@@ -11 +11 @@
 import java.awt.Toolkit;
 import java.awt.event.ActionEvent;
+import java.awt.event.KeyEvent;
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -209 +210 @@
      *
      * @param content Any element that can be displayed in the message dialog
-     * @param placeContentInScrollPane if  true, places  the content in a JScrollPane
+     * @param placeContentInScrollPane if true, places the content in a JScrollPane
      * @return {@code this}
      */
@@ -447 +448 @@
     private void setupEscListener() {
         Action actionListener = new AbstractAction() {
-            @Override public void actionPerformed(ActionEvent actionEvent) {
+            @Override
+            public void actionPerformed(ActionEvent actionEvent) {
                 // 0 means that the dialog has been closed otherwise.
                 // We need to set it to zero again, in case the dialog has been re-used
@@ -608 +610 @@
     /**
      * Convenience function that converts a given string into a JMultilineLabel
-     * @param msg
-     * @return JMultilineLabel
+     * @param msg the message to display
+     * @return JMultilineLabel displaying {@code msg}
      */
     private static JMultilineLabel string2label(String msg) {
@@ -616 +618 @@
         Dimension screenSize = Toolkit.getDefaultToolkit().getScreenSize();
         lbl.setMaxWidth(screenSize.width/2);
+        // Disable default Enter key binding to allow dialog's one (then enables to hit default button from here)
+        lbl.getInputMap().put(KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, 0), new Object());
         return lbl;
     }