Ignore:
Timestamp:
2009-10-06T23:52:41+02:00 (17 years ago)
Author:
Gubaer
Message:

Improved dialogs in the context of conflict resolution.
Improved support for online help in the context of conflict resolution, see also info about conflicts

Location:
trunk/src/org/openstreetmap/josm/actions
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/DownloadReferrersAction.java

    r2017 r2253  
    3535
    3636    public DownloadReferrersAction() {
    37         super(tr("Download referrers from OSM..."), "downloadreferrers", tr("Download primitives referring to one of the selected primitives"),
    38                 Shortcut.registerShortcut("file:downloadreferrers", tr("File: {0}", tr("Download referrers...")), KeyEvent.VK_D, Shortcut.GROUPS_ALT2+Shortcut.GROUP_HOTKEY), true);
     37        super(tr("Download parent ways/relations..."), "downloadreferrers", tr("Download primitives referring to one of the selected primitives"),
     38                Shortcut.registerShortcut("file:downloadreferrers", tr("File: {0}", tr("Download parent ways/relations ...")), KeyEvent.VK_D, Shortcut.GROUPS_ALT2+Shortcut.GROUP_HOTKEY), true);
    3939    }
    4040

  • trunk/src/org/openstreetmap/josm/actions/UpdateDataAction.java

    r2025 r2253  
    1717public class UpdateDataAction extends JosmAction{
    1818    public UpdateDataAction() {
    19         super(tr("Update Data"),
     19        super(tr("Update data"),
    2020                "updatedata",
    21                 tr("Updates the current data layer from the server (re-downloads data)"),
     21                tr("Updates the objects in the current data layer from the server "),
    2222                Shortcut.registerShortcut("file:updatedata",
    2323                        tr("Update Data"),

  • trunk/src/org/openstreetmap/josm/actions/UpdateSelectionAction.java

    r2077 r2253  
    8787     */
    8888    public UpdateSelectionAction() {
    89         super(tr("Update Selection"),
     89        super(tr("Update selections"),
    9090                "updateselection",
    91                 tr("Updates the currently selected primitives from the server"),
     91                tr("Updates the currently selected objects from the server (re-downloads data)"),
    9292                Shortcut.registerShortcut("file:updateselection",
    9393                        tr("Update Selection"),

  • trunk/src/org/openstreetmap/josm/actions/UploadAction.java

    r2250 r2253  
    55
    66import java.awt.event.ActionEvent;
     7import java.awt.event.ActionListener;
    78import java.awt.event.KeyEvent;
    89import java.io.IOException;
    … …  
    1718import java.util.regex.Pattern;
    1819
     20import javax.swing.JButton;
     21import javax.swing.JDialog;
    1922import javax.swing.JOptionPane;
    2023
    … …  
    3336import org.openstreetmap.josm.gui.ExceptionDialogUtil;
    3437import org.openstreetmap.josm.gui.PleaseWaitRunnable;
     38import org.openstreetmap.josm.gui.help.HelpBrowser;
     39import org.openstreetmap.josm.gui.help.HelpBuilder;
    3540import org.openstreetmap.josm.gui.io.UploadDialog;
    3641import org.openstreetmap.josm.gui.layer.OsmDataLayer;
    … …  
    4449import org.openstreetmap.josm.io.OsmTransferException;
    4550import org.openstreetmap.josm.tools.DateUtils;
     51import org.openstreetmap.josm.tools.ImageProvider;
    4652import org.openstreetmap.josm.tools.Shortcut;
     53import org.openstreetmap.josm.tools.WindowGeometry;
    4754import org.xml.sax.SAXException;
    4855
    … …  
    114121
    115122    public UploadAction() {
    116         super(tr("Upload to OSM..."), "upload", tr("Upload all changes to the OSM server."),
    117                 Shortcut.registerShortcut("file:upload", tr("File: {0}", tr("Upload to OSM...")), KeyEvent.VK_U, Shortcut.GROUPS_ALT1+Shortcut.GROUP_HOTKEY), true);
     123        super(tr("Upload data"), "upload", tr("Upload all changes in the current data layer to the OSM server"),
     124                Shortcut.registerShortcut("file:upload", tr("File: {0}", tr("Upload data")), KeyEvent.VK_U, Shortcut.GROUPS_ALT1+Shortcut.GROUP_HOTKEY), true);
    118125    }
    119126
    … …  
    222229     * @param myVersion  the version of the primitive in the local dataset
    223230     */
    224     protected void handleUploadConflictForKnownConflict(OsmPrimitiveType primitiveType, long id, String serverVersion, String myVersion) {
    225         Object[] options = new Object[] {
    226                 tr("Synchronize {0} {1} only", tr(primitiveType.getAPIName()), id),
    227                 tr("Synchronize entire dataset"),
    228                 tr("Cancel")
     231    protected void handleUploadConflictForKnownConflict(final OsmPrimitiveType primitiveType, final long id, String serverVersion, String myVersion) {
     232        JButton[] options = new JButton[] {
     233                new JButton(tr("Synchronize {0} {1} only", tr(primitiveType.getAPIName()), id)),
     234                new JButton(tr("Synchronize entire dataset")),
     235                new JButton(tr("Cancel")),
     236                new JButton(tr("Help"))
    229237        };
    230         Object defaultOption = options[0];
     238        options[0].setIcon(ImageProvider.get("updatedata"));
     239        options[1].setIcon(ImageProvider.get("updatedata"));
     240        options[2].setIcon(ImageProvider.get("cancel"));
     241        options[3].setIcon(ImageProvider.get("help"));
    231242        String msg =  tr("<html>Uploading <strong>failed</strong> because the server has a newer version of one<br>"
    232243                + "of your nodes, ways, or relations.<br>"
    … …  
    238249                + "Click <strong>{6}</strong> to abort and continue editing.<br></html>",
    239250                tr(primitiveType.getAPIName()), id, serverVersion, myVersion,
    240                 options[0], options[1], options[2]
    241         );
    242         int optionsType = JOptionPane.YES_NO_CANCEL_OPTION;
    243         int ret = JOptionPane.showOptionDialog(
    244                 null,
     251                options[0].getText(), options[1].getText(), options[2].getText()
     252        );
     253        final JOptionPane pane = new JOptionPane(
    245254                msg,
    246                 tr("Conflict detected"),
    247                 optionsType,
    248255                JOptionPane.ERROR_MESSAGE,
     256                JOptionPane.DEFAULT_OPTION,
    249257                null,
    250258                options,
    251                 defaultOption
    252         );
    253         switch(ret) {
    254             case JOptionPane.CLOSED_OPTION: return;
    255             case JOptionPane.CANCEL_OPTION: return;
    256             case 0: synchronizePrimitive(primitiveType, id); break;
    257             case 1: synchronizeDataSet(); break;
    258             default:
    259                 // should not happen
    260                 throw new IllegalStateException(tr("Unexpected return value. Got {0}.", ret));
    261         }
     259                options[0]
     260        );
     261        final JDialog dialog = new JDialog(
     262                JOptionPane.getFrameForComponent(Main.parent),
     263                tr("Conflicts detected"),
     264                true);
     265        options[0].addActionListener(
     266                new ActionListener() {
     267                    public void actionPerformed(ActionEvent e) {
     268                        dialog.setVisible(false);
     269                        synchronizePrimitive(primitiveType, id);
     270                    }
     271                }
     272        );
     273        options[1].addActionListener(
     274                new ActionListener() {
     275                    public void actionPerformed(ActionEvent e) {
     276                        dialog.setVisible(false);
     277                        synchronizeDataSet();
     278                    }
     279                }
     280        );
     281        options[2].addActionListener(
     282                new ActionListener() {
     283                    public void actionPerformed(ActionEvent e) {
     284                        dialog.setVisible(false);
     285                    }
     286                }
     287        );
     288        options[3].addActionListener(
     289                new ActionListener() {
     290                    public void actionPerformed(ActionEvent e) {
     291                        HelpBrowser b = new HelpBrowser();
     292                        b.setUrlForHelpTopic("Help/Concepts/Conflict");
     293                        b.setVisible(true);
     294                    }
     295                }
     296        );
     297        dialog.setContentPane(pane);
     298        dialog.pack();
     299        HelpBuilder.setHelpContext(dialog.getRootPane(), "Concepts/Conflict");
     300        WindowGeometry.centerOnScreen(dialog.getSize()).applySafe(dialog);
     301        dialog.setVisible(true);
    262302    }
    263303
    … …  
    268308     */
    269309    protected void handleUploadConflictForUnknownConflict() {
    270         Object[] options = new Object[] {
    271                 tr("Synchronize entire dataset"),
    272                 tr("Cancel")
     310        JButton[] options = new JButton[] {
     311                new JButton(tr("Synchronize entire dataset")),
     312                new JButton(tr("Cancel")),
     313                new JButton(tr("Help"))
    273314        };
    274315        Object defaultOption = options[0];
    … …  
    278319                + "Click <strong>{0}</strong> to synchronize the entire local dataset with the server.<br>"
    279320                + "Click <strong>{1}</strong> to abort and continue editing.<br></html>",
    280                 options[0], options[1]
    281         );
    282         int optionsType = JOptionPane.YES_NO_OPTION;
    283         int ret = JOptionPane.showOptionDialog(
    284                 null,
     321                options[0].getText(), options[1].getText()
     322        );
     323        final JOptionPane pane = new JOptionPane(
    285324                msg,
    286                 tr("Conflict detected"),
    287                 optionsType,
    288325                JOptionPane.ERROR_MESSAGE,
     326                JOptionPane.DEFAULT_OPTION,
    289327                null,
    290328                options,
    291                 defaultOption
    292         );
    293         switch(ret) {
    294             case JOptionPane.CLOSED_OPTION: return;
    295             case 1: return;
    296             case 0: synchronizeDataSet(); break;
    297             default:
    298                 // should not happen
    299                 throw new IllegalStateException(tr("Unexpected return value. Got {0}.", ret));
    300         }
     329                options[0]
     330        );
     331        final JDialog dialog = new JDialog(
     332                JOptionPane.getFrameForComponent(Main.parent),
     333                tr("Conflicts detected"),
     334                true);
     335
     336        options[0].addActionListener(
     337                new ActionListener() {
     338                    public void actionPerformed(ActionEvent e) {
     339                        dialog.setVisible(false);
     340                        synchronizeDataSet();
     341                    }
     342                }
     343        );
     344        options[1].addActionListener(
     345                new ActionListener() {
     346                    public void actionPerformed(ActionEvent e) {
     347                        dialog.setVisible(false);
     348                    }
     349                }
     350        );
     351        options[2].addActionListener(
     352                new ActionListener() {
     353                    public void actionPerformed(ActionEvent e) {
     354                        HelpBrowser b = new HelpBrowser();
     355                        b.setUrlForHelpTopic("Help/Concepts/Conflict");
     356                        b.setVisible(true);
     357                    }
     358                }
     359        );
     360        dialog.setContentPane(pane);
     361        dialog.pack();
     362        HelpBuilder.setHelpContext(dialog.getRootPane(), "Concepts/Conflict");
     363        WindowGeometry.centerOnScreen(dialog.getSize()).applySafe(dialog);
     364        dialog.setVisible(true);
    301365    }
    302366

  • trunk/src/org/openstreetmap/josm/actions/UploadSelectionAction.java

    r2250 r2253  
    4444    public UploadSelectionAction() {
    4545        super(
    46                 tr("Upload selection..."),
     46                tr("Upload selection"),
    4747                "uploadselection",
    48                 tr("Upload the current selection to the OSM server."),
     48                tr("Upload all changes in the current current selection to the OSM server."),
    4949                null, /* no shortcut */
    5050                true);

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTask.java

    r2215 r2253  
    2121import org.openstreetmap.josm.gui.layer.Layer;
    2222import org.openstreetmap.josm.gui.layer.OsmDataLayer;
    23 import org.openstreetmap.josm.gui.progress.NullProgressMonitor;
    2423import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    2524import org.openstreetmap.josm.io.BoundingBoxDownloader;
    … …  
    2928import org.openstreetmap.josm.tools.ExceptionUtil;
    3029import org.xml.sax.SAXException;
    31 
    3230
    3331/**

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTaskList.java

    r2097 r2253  
    166166        for (OsmPrimitive primitive : ds.relations) {
    167167            if (! primitive.incomplete && primitive.getId() == 0) {
    168                 ret.add(primitive);;
     168                ret.add(primitive);
    169169            }
    170170        }
    … …  
    232232        );
    233233        switch(ret) {
    234         case JOptionPane.CLOSED_OPTION: return;
    235         case JOptionPane.NO_OPTION: return;
    236         case JOptionPane.YES_OPTION: updatePotentiallyDeletedPrimitives(potentiallyDeleted); break;
     234            case JOptionPane.CLOSED_OPTION: return;
     235            case JOptionPane.NO_OPTION: return;
     236            case JOptionPane.YES_OPTION: updatePotentiallyDeletedPrimitives(potentiallyDeleted); break;
    237237        }
    238238    }
Note: See TracChangeset for help on using the changeset viewer.