Changeset 3310 in josm

Timestamp:

Jun 4, 2010 10:36:15 PM (3 years ago)

Author:

bastiK

Message:

fixed #4879 - tooltip: What does "guess" in correlate photos?, #5106 - Broken autocompletion (with numbers?)

Location:

trunk/src/org/openstreetmap/josm/gui

Files:

• layer/geoimage/CorrelateGpxWithImages.java (modified) (1 diff)
• layer/geoimage/GeoImageLayer.java (modified) (1 diff)
• tagging/ac/AutoCompletingComboBox.java (modified) (2 diffs)
• tagging/ac/AutoCompletingTextField.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java

@@ -513 +513 @@
 
         JButton buttonAutoGuess = new JButton(tr("Auto-Guess"));
+        buttonAutoGuess.setToolTipText(tr("Matches first photo with first gpx point"));
         buttonAutoGuess.addActionListener(new AutoGuessActionListener());
 

trunk/src/org/openstreetmap/josm/gui/layer/geoimage/GeoImageLayer.java

@@ -238 +238 @@
                 Main.map.mapView.addPropertyChangeListener(layer);
                 if (Main.map.getToggleDialog(ImageViewerDialog.class) == null) {
-                    System.err.println("JO");
-                    ImageViewerDialog.newInstance();// = new ImageViewerDialog();
+                    ImageViewerDialog.newInstance();
                     Main.map.addToggleDialog(ImageViewerDialog.getInstance());
                 }

trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletingComboBox.java

@@ -20 +20 @@
 import javax.swing.text.StyleConstants;
 
+import org.openstreetmap.josm.Main;
+
 /**
  * @author guilhem.bonnefille@gmail.com
@@ -69 +71 @@
             
             // if the text starts with a number we don't autocomplete
-            //
-            try {
-                Long.parseLong(str);
-                if (curText.length() == 0) {
-                    // we don't autocomplete on numbers
+            if (Main.pref.getBoolean("autocomplete.dont_complete_numbers", false)) {
+                try {
+                    Long.parseLong(str);
+                    if (curText.length() == 0) {
+                        // we don't autocomplete on numbers
+                        return;
+                    }
+                    Long.parseLong(curText);
                     return;
-                }
-                Long.parseLong(curText);
-                return;
-            } catch (NumberFormatException e) {
-                // either the new text or the current text isn't a number. We continue with
-                // autocompletion
+                } catch (NumberFormatException e) {
+                    // either the new text or the current text isn't a number. We continue with
+                    // autocompletion
+                }
             }
             

trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletingTextField.java

@@ -21 +21 @@
 import javax.swing.text.StyleConstants;
 
+import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.gui.tagging.ac.AutoCompletionList;
 import org.openstreetmap.josm.gui.util.TableCellEditorSupport;
@@ -71 +72 @@
             String currentText = getText(0, getLength());
             // if the text starts with a number we don't autocomplete
-            //
-            try {
-                Long.parseLong(str);
-                if (currentText.length() == 0) {
-                    // we don't autocomplete on numbers
+            if (Main.pref.getBoolean("autocomplete.dont_complete_numbers", false)) {
+                try {
+                    Long.parseLong(str);
+                    if (currentText.length() == 0) {
+                        // we don't autocomplete on numbers
+                        super.insertString(offs, str, a);
+                        return;
+                    }
+                    Long.parseLong(currentText);
                     super.insertString(offs, str, a);
                     return;
+                } catch(NumberFormatException e) {
+                    // either the new text or the current text isn't a number. We continue with
+                    // autocompletion
                 }
-                Long.parseLong(currentText);
-                super.insertString(offs, str, a);
-                return;
-            } catch(NumberFormatException e) {
-                // either the new text or the current text isn't a number. We continue with
-                // autocompletion
             }
             String prefix = currentText.substring(0, offs);