Changeset 7720 in josm

Timestamp:

2014-11-10T02:07:41+01:00 (10 years ago)

Author:

Don-vip

Message:

fix #10739 - Improve note input dialogs (patch by ToeBee)

Location:

trunk/src/org/openstreetmap/josm

Files:

• actions/mapmode/AddNoteAction.java (modified) (3 diffs)
• gui/NoteInputDialog.java (added)
• gui/dialogs/NoteDialog.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/actions/mapmode/AddNoteAction.java

@@ -6 +6 @@
 import java.awt.event.MouseEvent;
 
-import javax.swing.JLabel;
 import javax.swing.JOptionPane;
-import javax.swing.JScrollPane;
-import javax.swing.JTextArea;
 
 import org.openstreetmap.josm.Main;
@@ -15 +12 @@
 import org.openstreetmap.josm.data.osm.NoteData;
 import org.openstreetmap.josm.gui.MapFrame;
+import org.openstreetmap.josm.gui.NoteInputDialog;
 import org.openstreetmap.josm.gui.Notification;
 import org.openstreetmap.josm.gui.dialogs.NoteDialog;
@@ -63 +61 @@
         Main.map.selectMapMode(Main.map.mapModeSelect);
         LatLon latlon = Main.map.mapView.getLatLon(e.getPoint().x, e.getPoint().y);
-        JLabel label = new JLabel(tr("Enter a comment for a new note"));
-        JTextArea textArea = new JTextArea();
-        textArea.setRows(6);
-        textArea.setColumns(30);
-        textArea.setLineWrap(true);
-        JScrollPane scrollPane = new JScrollPane(textArea);
 
-        Object[] components = new Object[]{label, scrollPane};
-        int option = JOptionPane.showConfirmDialog(Main.map,
-                components,
-                tr("Create new note"),
-                JOptionPane.OK_CANCEL_OPTION,
-                JOptionPane.PLAIN_MESSAGE,
-                NoteDialog.ICON_NEW);
-        if (option == JOptionPane.OK_OPTION) {
-            String input = textArea.getText();
-            if (input != null && !input.isEmpty()) {
-                noteData.createNote(latlon, input);
-            } else {
-                Notification notification = new Notification("You must enter a comment to create a new note");
-                notification.setIcon(JOptionPane.WARNING_MESSAGE);
-                notification.show();
-            }
+        NoteInputDialog dialog = new NoteInputDialog(Main.parent, tr("Create new note"), tr("Create note"));
+        dialog.showNoteDialog(tr("Enter a detailed comment to create a note"), NoteDialog.ICON_NEW);
+
+        if (dialog.getValue() != 1) {
+            Main.debug("User aborted note creation");
+            return;
+        }
+        String input = dialog.getInputText();
+        if (input != null && !input.isEmpty()) {
+            noteData.createNote(latlon, input);
+        } else {
+            Notification notification = new Notification(tr("You must enter a comment to create a new note"));
+            notification.setIcon(JOptionPane.WARNING_MESSAGE);
+            notification.show();
         }
     }

trunk/src/org/openstreetmap/josm/gui/dialogs/NoteDialog.java

@@ -35 +35 @@
 import org.openstreetmap.josm.gui.MapView;
 import org.openstreetmap.josm.gui.MapView.LayerChangeListener;
+import org.openstreetmap.josm.gui.NoteInputDialog;
 import org.openstreetmap.josm.gui.SideButton;
 import org.openstreetmap.josm.gui.layer.Layer;
@@ -294 +295 @@
                 return;
             }
-            Object userInput = JOptionPane.showInputDialog(Main.map,
-                    tr("Add comment to note:"),
-                    tr("Add comment"),
-                    JOptionPane.QUESTION_MESSAGE,
-                    ICON_COMMENT,
-                    null,null);
-            if (userInput == null) { //user pressed cancel
+            NoteInputDialog dialog = new NoteInputDialog(Main.parent, tr("Comment on note"), tr("Add comment"));
+            dialog.showNoteDialog(tr("Add comment to note:"), NoteDialog.ICON_COMMENT);
+            if (dialog.getValue() != 1) {
+                Main.debug("User aborted note reopening");
                 return;
             }
-            noteData.addCommentToNote(note, userInput.toString());
+            noteData.addCommentToNote(note, dialog.getInputText());
         }
     }
@@ -317 +315 @@
         @Override
         public void actionPerformed(ActionEvent e) {
-            Object userInput = JOptionPane.showInputDialog(Main.map,
-                    tr("Close note with message:"),
-                    tr("Close Note"),
-                    JOptionPane.QUESTION_MESSAGE,
-                    ICON_CLOSED,
-                    null,null);
-            if (userInput == null) { //user pressed cancel
+            NoteInputDialog dialog = new NoteInputDialog(Main.parent, tr("Close note"), tr("Close note"));
+            dialog.showNoteDialog(tr("Close note with message:"), NoteDialog.ICON_CLOSED);
+            if (dialog.getValue() != 1) {
+                Main.debug("User aborted note closing");
                 return;
             }
             Note note = displayList.getSelectedValue();
-            noteData.closeNote(note, userInput.toString());
+            noteData.closeNote(note, dialog.getInputText());
         }
     }
@@ -358 +353 @@
         @Override
         public void actionPerformed(ActionEvent e) {
-            Object userInput = JOptionPane.showInputDialog(Main.map,
-                    tr("Reopen note with message:"),
-                    tr("Reopen note"),
-                    JOptionPane.QUESTION_MESSAGE,
-                    ICON_OPEN,
-                    null,null);
-            if (userInput == null) { //user pressed cancel
+            NoteInputDialog dialog = new NoteInputDialog(Main.parent, tr("Reopen note"), tr("Reopen note"));
+            dialog.showNoteDialog(tr("Reopen note with message:"), NoteDialog.ICON_OPEN);
+            if (dialog.getValue() != 1) {
+                Main.debug("User aborted note reopening");
                 return;
             }
+
             Note note = displayList.getSelectedValue();
-            noteData.reOpenNote(note, userInput.toString());
+            noteData.reOpenNote(note, dialog.getInputText());
         }
     }