Changeset 1847 in josm


Ignore:
Timestamp:
Jul 25, 2009 11:44:19 PM (4 years ago)
Author:
Gubaer
Message:

replaced calls to JOptionPane.* by calls to OptionPaneUtil.*

Location:
trunk/src/org/openstreetmap/josm
Files:
37 edited

Legend:

Unmodified
Added
Removed

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

    r1820 r1847  
    1313import org.openstreetmap.josm.Main; 
    1414import org.openstreetmap.josm.data.osm.OsmPrimitive; 
     15import org.openstreetmap.josm.gui.OptionPaneUtil; 
    1516import org.openstreetmap.josm.tools.OpenBrowser; 
    1617import org.openstreetmap.josm.tools.Shortcut; 
    … …  
    5859 
    5960        if (primitivesToShow.isEmpty()) { 
    60             JOptionPane.showMessageDialog( 
     61            OptionPaneUtil.showMessageDialog( 
    6162                    Main.parent, 
    6263                    tr("Please select at least one already uploaded node, way, or relation."), 

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

    r1846 r1847  
    2222import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    2323import org.openstreetmap.josm.data.osm.Way; 
     24import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2425import org.openstreetmap.josm.tools.Shortcut; 
    2526 
    … …  
    132133 
    133134        if (nodes.size() < 4) { 
    134             JOptionPane.showMessageDialog(Main.parent, tr("Please select at least four nodes.")); 
     135            OptionPaneUtil.showMessageDialog( 
     136                    Main.parent, 
     137                    tr("Please select at least four nodes."), 
     138                    tr("Information"), 
     139                    JOptionPane.INFORMATION_MESSAGE 
     140            ); 
    135141            return; 
    136142        } 

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

    r1820 r1847  
    1818import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    1919import org.openstreetmap.josm.data.osm.Way; 
     20import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2021import org.openstreetmap.josm.tools.Shortcut; 
    2122 
    … …  
    6061        } 
    6162        if (nodes.size() < 3) { 
    62             JOptionPane.showMessageDialog(Main.parent, tr("Please select at least three nodes.")); 
     63            OptionPaneUtil.showMessageDialog( 
     64                    Main.parent, 
     65                    tr("Please select at least three nodes."), 
     66                    tr("Information"), 
     67                    JOptionPane.INFORMATION_MESSAGE 
     68            ); 
    6369            return; 
    6470        } 

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

    r1843 r1847  
    1515import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    1616import org.openstreetmap.josm.data.osm.Way; 
     17import org.openstreetmap.josm.gui.OptionPaneUtil; 
    1718import org.openstreetmap.josm.io.OsmApi; 
    1819import org.openstreetmap.josm.io.OsmApiInitializationException; 
    … …  
    4748                total = add.size() + update.size() + delete.size(); 
    4849                if(total > maxElements) { 
    49                     JOptionPane.showMessageDialog( 
     50                    OptionPaneUtil.showMessageDialog( 
    5051                            Main.parent, 
    5152                            tr("Current number of changes exceeds the max. number of changes, current is {0}, max is {1}", 
    … …  
    6162        } catch (OsmApiInitializationException e) { 
    6263            e.printStackTrace(); 
    63             JOptionPane.showMessageDialog( 
     64            OptionPaneUtil.showMessageDialog( 
    6465                    Main.parent, 
    6566                    tr("Failed to initialize API. Please try again later."), 
    … …  
    8889                        continue; 
    8990                    } 
    90                     JOptionPane.showMessageDialog(Main.parent, 
     91                    OptionPaneUtil.showMessageDialog(Main.parent, 
    9192                            tr("Length of value for tag ''{0}'' on primitive {1} exceeds the max. allowed length {2}. Values length is {3}.", 
    9293                                    e.getKey(), Long.toString(osmPrimitive.id), 255, e.getValue().length() 
    … …  
    104105            if (osmPrimitive instanceof Way && 
    105106                    ((Way)osmPrimitive).nodes.size() > maxNodes) { 
    106                 JOptionPane.showMessageDialog( 
     107                OptionPaneUtil.showMessageDialog( 
    107108                        Main.parent, 
    108109                        tr("{0} nodes in way {1} exceed the max. allowed number of nodes {2}", 

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

    r1820 r1847  
    1616import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    1717import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor; 
     18import org.openstreetmap.josm.gui.OptionPaneUtil; 
    1819import org.openstreetmap.josm.gui.layer.Layer; 
    1920import org.openstreetmap.josm.tools.Shortcut; 
    … …  
    8687            } 
    8788            if (sel.isEmpty()) { 
    88                 JOptionPane.showMessageDialog(Main.parent, 
    89                         mode.equals("selection") ? tr("Nothing selected to zoom to.") : tr("No conflicts to zoom to")); 
     89                OptionPaneUtil.showMessageDialog( 
     90                        Main.parent, 
     91                        (mode.equals("selection") ? tr("Nothing selected to zoom to.") : tr("No conflicts to zoom to")), 
     92                        tr("Information"), 
     93                        JOptionPane.INFORMATION_MESSAGE 
     94                ); 
    9095                return null; 
    9196            } 

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

    r1820 r1847  
    3737import org.openstreetmap.josm.data.osm.Way; 
    3838import org.openstreetmap.josm.gui.ExtendedDialog; 
     39import org.openstreetmap.josm.gui.OptionPaneUtil; 
    3940import org.openstreetmap.josm.tools.GBC; 
    4041import org.openstreetmap.josm.tools.Pair; 
    … …  
    6566 
    6667        if (selectedWays.size() < 2) { 
    67             JOptionPane.showMessageDialog(Main.parent, tr("Please select at least two ways to combine.")); 
     68            OptionPaneUtil.showMessageDialog( 
     69                    Main.parent, 
     70                    tr("Please select at least two ways to combine."), 
     71                    tr("Information"), 
     72                    JOptionPane.INFORMATION_MESSAGE 
     73            ); 
    6874            return; 
    6975        } 
    … …  
    152158                nodeList = (List<Node>) secondTry; 
    153159            } else { 
    154                 JOptionPane.showMessageDialog(Main.parent, secondTry); 
     160                OptionPaneUtil.showMessageDialog( 
     161                        Main.parent, 
     162                        secondTry, // FIXME: not sure whether this fits in a dialog 
     163                        tr("Information"), 
     164                        JOptionPane.INFORMATION_MESSAGE 
     165                ); 
    155166                return; 
    156167            } 

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

    r1820 r1847  
    2424import org.openstreetmap.josm.data.osm.Way; 
    2525import org.openstreetmap.josm.data.osm.visitor.AbstractVisitor; 
     26import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2627import org.openstreetmap.josm.tools.Shortcut; 
    2728 
    … …  
    136137        Collection<OsmPrimitive> sel = getCurrentDataSet().getSelected(); 
    137138        if (sel.isEmpty()) { 
    138             JOptionPane.showMessageDialog( 
     139            OptionPaneUtil.showMessageDialog( 
    139140                    Main.parent, 
    140                     tr("Please select something to copy.") 
     141                    tr("Please select something to copy."), 
     142                    tr("Information"), 
     143                    JOptionPane.INFORMATION_MESSAGE 
    141144            ); 
    142145            return true; 

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

    r1846 r1847  
    2323import org.openstreetmap.josm.data.osm.Way; 
    2424import org.openstreetmap.josm.data.osm.visitor.CollectBackReferencesVisitor; 
     25import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2526import org.openstreetmap.josm.tools.Shortcut; 
    2627 
    … …  
    202203 
    203204            if (sUnder == 0) { 
    204                 JOptionPane.showMessageDialog(Main.parent, tr("Those nodes are not in a circle.")); 
     205                OptionPaneUtil.showMessageDialog( 
     206                        Main.parent, 
     207                        tr("Those nodes are not in a circle. Aborting."), 
     208                        tr("Warning"), 
     209                        JOptionPane.WARNING_MESSAGE 
     210                ); 
    205211                return; 
    206212            } 
    … …  
    261267 
    262268        } else { 
    263             JOptionPane.showMessageDialog(Main.parent, tr("Please select exactly two or three nodes or one way with exactly two or three nodes.")); 
     269            OptionPaneUtil.showMessageDialog( 
     270                    Main.parent, 
     271                    tr("Please select exactly two or three nodes or one way with exactly two or three nodes."), 
     272                    tr("Information"), 
     273                    JOptionPane.INFORMATION_MESSAGE 
     274            ); 
    264275            return; 
    265276        } 

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

    r1820 r1847  
    1818import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    1919import org.openstreetmap.josm.data.osm.Way; 
     20import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2021import org.openstreetmap.josm.tools.Shortcut; 
    2122 
    … …  
    5960 
    6061        if (nodes.size() < 3) { 
    61             JOptionPane.showMessageDialog(Main.parent, tr("Please select at least three nodes.")); 
     62            OptionPaneUtil.showMessageDialog( 
     63                    Main.parent, 
     64                    tr("Please select at least three nodes."), 
     65                    tr("Information"), 
     66                    JOptionPane.INFORMATION_MESSAGE 
     67            ); 
    6268            return; 
    6369        } 

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

    r1811 r1847  
    1313 
    1414import org.openstreetmap.josm.Main; 
     15import org.openstreetmap.josm.gui.OptionPaneUtil; 
    1516import org.openstreetmap.josm.gui.download.DownloadDialog; 
    1617import org.openstreetmap.josm.gui.download.DownloadDialog.DownloadTask; 
    … …  
    3233    public DownloadAction() { 
    3334        super(tr("Download from OSM..."), "download", tr("Download map data from the OSM server."), 
    34         Shortcut.registerShortcut("file:download", tr("File: {0}", tr("Download from OSM...")), KeyEvent.VK_D, Shortcut.GROUPS_ALT1+Shortcut.GROUP_HOTKEY), true); 
     35                Shortcut.registerShortcut("file:download", tr("File: {0}", tr("Download from OSM...")), KeyEvent.VK_D, Shortcut.GROUPS_ALT1+Shortcut.GROUP_HOTKEY), true); 
    3536    } 
    3637 
    … …  
    4647        dialog.setOptionPane(pane); 
    4748 
    48         if (dlg.getWidth() > 1000) 
     49        if (dlg.getWidth() > 1000) { 
    4950            dlg.setSize(1000, dlg.getHeight()); 
    50         if (dlg.getHeight() > 600) 
     51        } 
     52        if (dlg.getHeight() > 600) { 
    5153            dlg.setSize(dlg.getWidth(),600); 
     54        } 
    5255 
    5356        boolean finish = false; 
    … …  
    6467                    } 
    6568                } 
    66             } else 
     69            } else { 
    6770                finish = true; 
    68             if (!finish) 
    69                 JOptionPane.showMessageDialog(Main.parent, tr("Please select at least one task to download")); 
     71            } 
     72            if (!finish) { 
     73                OptionPaneUtil.showMessageDialog( 
     74                        Main.parent, 
     75                        tr("Please select at least one task to download"), 
     76                        tr("Error"), 
     77                        JOptionPane.ERROR_MESSAGE 
     78                ); 
     79            } 
    7080        } 
    7181 
    72                 dialog = null; 
     82        dialog = null; 
    7383        dlg.dispose(); 
    7484    } 

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

    r1820 r1847  
    1818import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    1919import org.openstreetmap.josm.data.osm.visitor.MergeVisitor; 
     20import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2021import org.openstreetmap.josm.gui.PleaseWaitRunnable; 
    2122import org.openstreetmap.josm.gui.layer.Layer; 
    … …  
    8586                msg = lastException.toString(); 
    8687            } 
    87             JOptionPane.showMessageDialog( 
     88            OptionPaneUtil.showMessageDialog( 
    8889                    Main.map, 
    8990                    msg, 
    … …  
    121122                return; 
    122123            Main.map.mapView.getEditLayer().getConflicts().add(visitor.getConflicts()); 
    123             JOptionPane op = new JOptionPane( 
     124            OptionPaneUtil.showMessageDialog( 
     125                    Main.parent, 
    124126                    tr("There were {0} conflicts during import.", 
    125                             visitor.getConflicts().size()), 
    126                             JOptionPane.WARNING_MESSAGE 
     127                            visitor.getConflicts().size() 
     128                    ), 
     129                    tr("Conflicts during download"), 
     130                    JOptionPane.WARNING_MESSAGE 
    127131            ); 
    128             JDialog dialog = op.createDialog(null, tr("Conflicts in data")); 
    129             dialog.setAlwaysOnTop(true); 
    130             dialog.setModal(true); 
    131             dialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE); 
    132             dialog.setVisible(true); 
    133132        } 
    134133 

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

    r1820 r1847  
    2929import org.openstreetmap.josm.data.gpx.GpxData; 
    3030import org.openstreetmap.josm.gui.ExtendedDialog; 
     31import org.openstreetmap.josm.gui.OptionPaneUtil; 
    3132import org.openstreetmap.josm.gui.layer.GpxLayer; 
    3233import org.openstreetmap.josm.gui.layer.Layer; 
    … …  
    6263        GpxLayer layer = getLayer(); 
    6364        if (layer == null) { 
    64             JOptionPane.showMessageDialog(Main.parent, tr("Nothing to export. Get some data first.")); 
     65            OptionPaneUtil.showMessageDialog( 
     66                    Main.parent, 
     67                    tr("Nothing to export. Get some data first."), 
     68                    tr("Information"), 
     69                    JOptionPane.INFORMATION_MESSAGE 
     70            ); 
    6571            return; 
    6672        } 

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

    r1820 r1847  
    3030 
    3131import org.openstreetmap.josm.Main; 
     32import org.openstreetmap.josm.gui.OptionPaneUtil; 
    3233import org.openstreetmap.josm.tools.ImageProvider; 
    3334import org.openstreetmap.josm.tools.LanguageInfo; 
    … …  
    110111        } else if (tr("Edit").equals(e.getActionCommand())) { 
    111112            if (!url.startsWith(baseurl)) { 
    112                 JOptionPane.showMessageDialog(Main.parent, tr("Can only edit help pages from JOSM Online Help")); 
     113                OptionPaneUtil.showMessageDialog( 
     114                        Main.parent, 
     115                        tr("Can only edit help pages from JOSM Online Help"), 
     116                        tr("Warning"), 
     117                        JOptionPane.WARNING_MESSAGE 
     118                ); 
    113119                return; 
    114120            } 

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

    r1820 r1847  
    7272 
    7373        if (selectedNodes.size() < 2) { 
    74             JOptionPane.showMessageDialog(Main.parent, tr("Please select at least two nodes to merge.")); 
     74            JOptionPane.showMessageDialog( 
     75                    Main.parent, 
     76                    tr("Please select at least two nodes to merge."), 
     77                    tr("Warning"), 
     78                    JOptionPane.WARNING_MESSAGE 
     79            ); 
    7580            return; 
    7681        } 

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

    r1820 r1847  
    4949 
    5050        if (nodes.size() == 0) { 
    51             JOptionPane.showMessageDialog(Main.parent, tr("Please select at least one node or way.")); 
     51            JOptionPane.showMessageDialog( 
     52                    Main.parent, 
     53                    tr("Please select at least one node or way."), 
     54                    tr("Information"), 
     55                    JOptionPane.INFORMATION_MESSAGE 
     56            ); 
    5257            return; 
    5358        } 

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

    r1820 r1847  
    1717import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    1818import org.openstreetmap.josm.data.osm.visitor.AllNodesVisitor; 
     19import org.openstreetmap.josm.gui.OptionPaneUtil; 
    1920import org.openstreetmap.josm.tools.Shortcut; 
    2021 
    … …  
    102103                // Revert move 
    103104                ((MoveCommand) c).moveAgain(-distx, -disty); 
    104                 JOptionPane.showMessageDialog(Main.parent, 
    105                         tr("Cannot move objects outside of the world.")); 
     105                OptionPaneUtil.showMessageDialog( 
     106                        Main.parent, 
     107                        tr("Cannot move objects outside of the world."), 
     108                        tr("Warning"), 
     109                        JOptionPane.WARNING_MESSAGE 
     110                ); 
    106111                return; 
    107112            } 

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

    r1637 r1847  
    1313 
    1414import org.openstreetmap.josm.Main; 
     15import org.openstreetmap.josm.gui.OptionPaneUtil; 
    1516import org.openstreetmap.josm.io.FileImporter; 
    1617import org.openstreetmap.josm.tools.Shortcut; 
    … …  
    2930    public OpenFileAction() { 
    3031        super(tr("Open..."), "open", tr("Open a file."), 
    31         Shortcut.registerShortcut("system:open", tr("File: {0}", tr("Open...")), KeyEvent.VK_O, Shortcut.GROUP_MENU)); 
     32                Shortcut.registerShortcut("system:open", tr("File: {0}", tr("Open...")), KeyEvent.VK_O, Shortcut.GROUP_MENU)); 
    3233    } 
    3334 
    … …  
    3738            return; 
    3839        File[] files = fc.getSelectedFiles(); 
    39         for (int i = files.length; i > 0; --i) 
     40        for (int i = files.length; i > 0; --i) { 
    4041            openFile(files[i-1]); 
     42        } 
    4143    } 
    4244 
    … …  
    4850            System.out.println("Open file: " + file.getAbsolutePath() + " (" + file.length() + " bytes)"); 
    4951            for (FileImporter importer : ExtensionFileFilter.importers) 
    50                 if (importer.acceptFile(file)) 
     52                if (importer.acceptFile(file)) { 
    5153                    importer.importData(file); 
     54                } 
    5255        } catch (IOException x) { 
    5356            x.printStackTrace(); 
    54             JOptionPane.showMessageDialog(Main.parent, tr("Could not read \"{0}\"", file.getName()) + "\n" 
    55                     + x.getMessage()); 
     57            OptionPaneUtil.showMessageDialog( 
     58                    Main.parent, 
     59                    tr("<html>Could not read file ''{0}\''. Error is: <br>{1}</html>", file.getName(), x.getMessage()), 
     60                    tr("Error"), 
     61                    JOptionPane.ERROR_MESSAGE 
     62            ); 
     63 
    5664        } 
    5765    } 

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

    r1838 r1847  
    1212 
    1313import javax.swing.JOptionPane; 
     14import javax.swing.text.html.Option; 
    1415 
    1516import org.openstreetmap.josm.Main; 
    … …  
    2324import org.openstreetmap.josm.data.osm.Way; 
    2425import org.openstreetmap.josm.gui.ConditionalOptionPaneUtil; 
     26import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2527import org.openstreetmap.josm.tools.Shortcut; 
    2628 
    … …  
    5759            if(osm instanceof Node) { 
    5860                if(dirnodes.size() == 2) { 
    59                     JOptionPane.showMessageDialog(Main.parent, tr("Only two nodes allowed")); 
     61                    OptionPaneUtil.showMessageDialog( 
     62                            Main.parent, 
     63                            tr("Only two nodes allowed"), 
     64                            tr("Information"), 
     65                            JOptionPane.INFORMATION_MESSAGE 
     66                    ); 
    6067                    return; 
    6168                } 
    … …  
    6572            // Check if selection consists now only of ways 
    6673            if (!(osm instanceof Way)) { 
    67                 JOptionPane.showMessageDialog(Main.parent, tr("Selection must consist only of ways.")); 
     74                OptionPaneUtil.showMessageDialog( 
     75                        Main.parent, 
     76                        tr("Selection must consist only of ways."), 
     77                        tr("Information"), 
     78                        JOptionPane.INFORMATION_MESSAGE 
     79                ); 
    6880                return; 
    6981            } 
    … …  
    7284            Way way = (Way)osm; 
    7385            if ((way.nodes.size() < 5) || (!way.nodes.get(0).equals(way.nodes.get(way.nodes.size() - 1)))) { 
    74                 JOptionPane.showMessageDialog(Main.parent, tr("Please select one or more closed ways of at least four nodes.")); 
     86                OptionPaneUtil.showMessageDialog( 
     87                        Main.parent, 
     88                        tr("Please select one or more closed ways of at least four nodes."), 
     89                        tr("Information"), 
     90                        JOptionPane.INFORMATION_MESSAGE 
     91                ); 
    7592                return; 
    7693            } 
    … …  
    112129        // Check, if selection held neither none nor two nodes 
    113130        if(dirnodes.size() == 1) { 
    114             JOptionPane.showMessageDialog(Main.parent, tr("Only one node selected")); 
     131            OptionPaneUtil.showMessageDialog( 
     132                    Main.parent, 
     133                    tr("Only one node selected"), 
     134                    tr("Warning"), 
     135                    JOptionPane.WARNING_MESSAGE 
     136            ); 
    115137            return; 
    116138        } 

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

    r1843 r1847  
    103103        /* only enable if there is something selected to paste into and 
    104104            if we don't have conflicting keys in the pastebuffer */ 
    105         setEnabled(selection != null && 
    106                 ! selection.isEmpty() && 
    107                 ! pasteBuffer.allPrimitives().isEmpty() && 
    108                 (getCurrentDataSet().getSelectedNodes().isEmpty() || 
    109                         ! containsSameKeysWithDifferentValues(pasteBuffer.nodes)) && 
    110                         (getCurrentDataSet().getSelectedWays().isEmpty() || 
    111                                 ! containsSameKeysWithDifferentValues(pasteBuffer.ways)) && 
    112                                 (getCurrentDataSet().getSelectedRelations().isEmpty() || 
    113                                         ! containsSameKeysWithDifferentValues(pasteBuffer.relations))); 
     105        DataSet ds = getCurrentDataSet(); 
     106        if (ds == null || ds.getSelected().isEmpty() || pasteBuffer == null || pasteBuffer.allPrimitives().isEmpty()) { 
     107            setEnabled(false); 
     108            return; 
     109        } 
     110        setEnabled((!ds.getSelectedNodes().isEmpty() && ! containsSameKeysWithDifferentValues(pasteBuffer.nodes)) || 
     111                (!ds.getSelectedWays().isEmpty() && ! containsSameKeysWithDifferentValues(pasteBuffer.ways)) || 
     112                (! ds.getSelectedRelations().isEmpty() && ! containsSameKeysWithDifferentValues(pasteBuffer.relations))); 
    114113    } 
    115114 
    116115    @Override public void pasteBufferChanged(DataSet newPasteBuffer) { 
    117         possiblyEnable(getCurrentDataSet().getSelected(), newPasteBuffer); 
     116        updateEnabledState(); 
    118117    } 
    119118 

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

    r1646 r1847  
    1515 
    1616import org.openstreetmap.josm.Main; 
     17import org.openstreetmap.josm.gui.OptionPaneUtil; 
    1718import org.openstreetmap.josm.gui.layer.Layer; 
    1819import org.openstreetmap.josm.tools.ImageProvider; 
    … …  
    5253            filerename.setEnabled(file != null); 
    5354        } 
    54         if (filerename.isEnabled()) 
     55        if (filerename.isEnabled()) { 
    5556            filerename.setSelected(Main.pref.getBoolean("layer.rename-file", true)); 
     57        } 
    5658 
    5759        final JOptionPane optionPane = new JOptionPane(panel, JOptionPane.QUESTION_MESSAGE, JOptionPane.OK_CANCEL_OPTION){ 
    … …  
    6264        }; 
    6365        final JDialog dlg = optionPane.createDialog(Main.parent, tr("Rename layer")); 
     66        dlg.setModal(true); 
     67        dlg.setAlwaysOnTop(true); 
     68        dlg.toFront(); 
    6469        dlg.setVisible(true); 
    6570 
    6671        Object answer = optionPane.getValue(); 
    6772        if (answer == null || answer == JOptionPane.UNINITIALIZED_VALUE || 
    68                 (answer instanceof Integer && (Integer)answer != JOptionPane.OK_OPTION)) { 
     73                (answer instanceof Integer && (Integer)answer != JOptionPane.OK_OPTION)) 
    6974            return; 
    70         } 
    7175 
    7276        String nameText = name.getText(); 
    … …  
    7579            if (filerename.isSelected()) { 
    7680                String newname = nameText; 
    77                 if (newname.indexOf("/") == -1 && newname.indexOf("\\") == -1) 
     81                if (newname.indexOf("/") == -1 && newname.indexOf("\\") == -1) { 
    7882                    newname = file.getParent() + File.separator + newname; 
     83                } 
    7984                String oldname = file.getName(); 
    80                 if (name.getText().indexOf('.') == -1 && oldname.indexOf('.') >= 0) 
     85                if (name.getText().indexOf('.') == -1 && oldname.indexOf('.') >= 0) { 
    8186                    newname += oldname.substring(oldname.lastIndexOf('.')); 
     87                } 
    8288                File newFile = new File(newname); 
    8389                if (file.renameTo(newFile)) { 
    … …  
    8591                    nameText = newFile.getName(); 
    8692                } else { 
    87                     JOptionPane.showMessageDialog(Main.parent, tr("Could not rename the file \"{0}\".", file.getPath())); 
     93                    OptionPaneUtil.showMessageDialog( 
     94                            Main.parent, 
     95                            tr("Could not rename file ''{0}''", file.getPath()), 
     96                            tr("Error"), 
     97                            JOptionPane.ERROR_MESSAGE 
     98                    ); 
    8899                    return; 
    89100                } 

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

    r1844 r1847  
    2121import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    2222import org.openstreetmap.josm.data.osm.Way; 
     23import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2324import org.openstreetmap.josm.tools.Shortcut; 
    2425 
    … …  
    4344        } 
    4445        if (sel.isEmpty()) { 
    45             JOptionPane.showMessageDialog(Main.parent, 
    46                     tr("Please select at least one way.")); 
     46            OptionPaneUtil.showMessageDialog( 
     47                    Main.parent, 
     48                    tr("Please select at least one way."), 
     49                    tr("Information"), 
     50                    JOptionPane.INFORMATION_MESSAGE 
     51            ); 
    4752            return; 
    4853        } 

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

    r1820 r1847  
    2323import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    2424import org.openstreetmap.josm.gui.ExtendedDialog; 
     25import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2526import org.openstreetmap.josm.gui.layer.GpxLayer; 
    2627import org.openstreetmap.josm.gui.layer.Layer; 
    … …  
    9091        } 
    9192        if (Main.map == null) { 
    92             JOptionPane.showMessageDialog(Main.parent, tr("No document open so nothing to save.")); 
     93            JOptionPane.showMessageDialog( 
     94                    Main.parent, 
     95                    tr("No document open so nothing to save."), 
     96                    tr("Warning"), 
     97                    JOptionPane.WARNING_MESSAGE 
     98            ); 
    9399            return false; 
    94100        } 
    … …  
    128134            dstStream = new FileOutputStream(dst); 
    129135        } catch (FileNotFoundException e) { 
    130             JOptionPane.showMessageDialog(Main.parent, tr("Could not back up file.")+"\n"+e.getMessage()); 
     136            OptionPaneUtil.showMessageDialog( 
     137                    Main.parent, 
     138                    tr("Could not back up file. Exception is: {0}", e.getMessage()), 
     139                    tr("Error"), 
     140                    JOptionPane.ERROR_MESSAGE 
     141            ); 
    131142            return; 
    132143        } 
    … …  
    192203                } 
    193204            } else { 
    194                 JOptionPane.showMessageDialog(Main.parent, tr("Unknown file extension.")); 
     205                OptionPaneUtil.showMessageDialog( 
     206                        Main.parent, 
     207                        tr("Unknown file extension for file ''{0}''", file.toString()), 
     208                        tr("Error"), 
     209                        JOptionPane.ERROR_MESSAGE 
     210                ); 
    195211                return; 
    196212            } 
    … …  
    198214        } catch (IOException e) { 
    199215            e.printStackTrace(); 
    200             JOptionPane.showMessageDialog(Main.parent, tr("An error occurred while saving.")+"\n"+e.getMessage()); 
     216            OptionPaneUtil.showMessageDialog( 
     217                    Main.parent, 
     218                    tr("<html>An error occurred while saving. <br>Error is: <br>{0}</html>", e.getMessage()), 
     219                    tr("Error"), 
     220                    JOptionPane.ERROR_MESSAGE 
     221            ); 
    201222 
    202223            try { 
    … …  
    208229            } catch (IOException e2) { 
    209230                e2.printStackTrace(); 
    210                 JOptionPane.showMessageDialog(Main.parent, tr("An error occurred while restoring backup file.")+"\n"+e2.getMessage()); 
     231                OptionPaneUtil.showMessageDialog( 
     232                        Main.parent, 
     233                        tr("<html>An error occurred while restoring backup file.<br> Error is: <br>{0}</html>", e2.getMessage()), 
     234                        tr("Error"), 
     235                        JOptionPane.ERROR_MESSAGE 
     236                ); 
    211237            } 
    212238        } 
    … …  
    235261                } 
    236262            } else { 
    237                 JOptionPane.showMessageDialog(Main.parent, tr("Unknown file extension.")); 
     263                OptionPaneUtil.showMessageDialog( 
     264                        Main.parent, 
     265                        tr("Unknown file extension."), 
     266                        tr("Error"), 
     267                        JOptionPane.ERROR_MESSAGE 
     268                ); 
    238269                return; 
    239270            } 
    … …  
    250281        } catch (IOException e) { 
    251282            e.printStackTrace(); 
    252             JOptionPane.showMessageDialog(Main.parent, tr("An error occurred while restoring backup file.")+"\n"+e.getMessage()); 
     283            OptionPaneUtil.showMessageDialog( 
     284                    Main.parent, 
     285                    tr("<html>An error occurred while restoring backup file.<br>Error is:<br>{0}</html>", e.getMessage()), 
     286                    tr("Error"), 
     287                    JOptionPane.ERROR_MESSAGE 
     288            );; 
    253289        } 
    254290    } 

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

    r1811 r1847  
    2020import org.openstreetmap.josm.data.osm.DataSet; 
    2121import org.openstreetmap.josm.data.osm.OsmPrimitive; 
     22import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2223import org.openstreetmap.josm.gui.download.DownloadDialog.DownloadTask; 
    2324import org.openstreetmap.josm.gui.layer.Layer; 
    … …  
    100101 
    101102        if(! errors.equals("")) { 
    102             JOptionPane.showMessageDialog(Main.parent, 
    103                     tr("The following errors occurred during mass download:") + "\r\n" + errors, 
     103            OptionPaneUtil.showMessageDialog( 
     104                    Main.parent, 
     105                    tr("<html>The following errors occurred during mass download: {0}</html>", errors), 
    104106                    tr("Errors during Download"), 
    105107                    JOptionPane.ERROR_MESSAGE); 
    … …  
    167169        ); 
    168170 
    169         int ret = JOptionPane.showOptionDialog( 
     171        int ret =OptionPaneUtil.showOptionDialog( 
    170172                Main.parent, 
    171173                message, 
    … …  
    173175                JOptionPane.YES_NO_OPTION, 
    174176                JOptionPane.WARNING_MESSAGE, 
    175                 null, 
    176177                options, 
    177178                options[0] 

  • trunk/src/org/openstreetmap/josm/actions/mapmode/SelectAction.java

    r1844 r1847  
    4141import org.openstreetmap.josm.gui.MapFrame; 
    4242import org.openstreetmap.josm.gui.MapView; 
     43import org.openstreetmap.josm.gui.OptionPaneUtil; 
    4344import org.openstreetmap.josm.gui.SelectionManager; 
    4445import org.openstreetmap.josm.gui.SelectionManager.SelectionEnded; 
    … …  
    246247                        ((MoveCommand) c).moveAgain(-dx, -dy); 
    247248 
    248                         JOptionPane.showMessageDialog(Main.parent, 
    249                                 tr("Cannot move objects outside of the world.")); 
     249                        OptionPaneUtil.showMessageDialog( 
     250                                Main.parent, 
     251                                tr("Cannot move objects outside of the world."), 
     252                                tr("Warning"), 
     253                                JOptionPane.WARNING_MESSAGE 
     254 
     255                        ); 
    250256                        restoreCursor(); 
    251257                        return; 

  • trunk/src/org/openstreetmap/josm/actions/search/SearchAction.java

    r1820 r1847  
    2323import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    2424import org.openstreetmap.josm.gui.ExtendedDialog; 
     25import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2526import org.openstreetmap.josm.tools.GBC; 
    2627import org.openstreetmap.josm.tools.Shortcut; 
    … …  
    4748            return; 
    4849        if (Main.map == null) { 
    49             JOptionPane.showMessageDialog(Main.parent, tr("No data loaded.")); 
     50            OptionPaneUtil.showMessageDialog( 
     51                    Main.parent, 
     52                    tr("Can't search because there is no loaded data."), 
     53                    tr("Warning"), 
     54                    JOptionPane.WARNING_MESSAGE 
     55            ); 
    5056            return; 
    5157        } 
    … …  
    190196                } 
    191197                Main.map.statusLine.setHelpText(msg); 
    192                 JOptionPane.showMessageDialog(Main.parent, msg); 
     198                OptionPaneUtil.showMessageDialog( 
     199                        Main.parent, 
     200                        msg, 
     201                        tr("Warning"), 
     202                        JOptionPane.WARNING_MESSAGE 
     203                ); 
    193204            } else { 
    194205                Main.map.statusLine.setHelpText(tr("Found {0} matches", foundMatches)); 
    195206            } 
    196207        } catch (SearchCompiler.ParseError e) { 
    197             JOptionPane.showMessageDialog(Main.parent, e.getMessage()); 
     208            OptionPaneUtil.showMessageDialog( 
     209                    Main.parent, 
     210                    e.getMessage(), 
     211                    tr("Error"), 
     212                    JOptionPane.ERROR_MESSAGE 
     213 
     214            ); 
    198215        } 
    199216    } 

  • trunk/src/org/openstreetmap/josm/gui/conflict/properties/PropertiesMergeModel.java

    r1811 r1847  
    2727import org.openstreetmap.josm.data.osm.RelationMember; 
    2828import org.openstreetmap.josm.data.osm.Way; 
     29import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2930import org.openstreetmap.josm.gui.conflict.MergeDecisionType; 
    3031import org.openstreetmap.josm.gui.progress.NullProgressMonitor; 
    … …  
    417418        String msg = e.getMessage() != null ? e.getMessage() : e.toString(); 
    418419        msg = msg.replaceAll("&", "&amp;").replaceAll("<", "&lt;").replaceAll(">", "&gt;"); 
    419         JOptionPane.showMessageDialog( 
     420        OptionPaneUtil.showMessageDialog( 
    420421                Main.parent, 
    421422                tr("<html>An error occurred while communicating with the server<br>" 
    … …  
    468469                tr("No, cancel operation") 
    469470        }; 
    470         int ret = JOptionPane.showOptionDialog( 
     471        int ret = OptionPaneUtil.showOptionDialog( 
    471472                Main.parent, 
    472473                tr("<html>There are {0} additional nodes used by way {1}<br>" 
    … …  
    478479                        JOptionPane.YES_NO_OPTION, 
    479480                        JOptionPane.QUESTION_MESSAGE, 
    480                         null, 
    481481                        options, 
    482482                        options[0] 
    … …  
    497497                tr("No, cancel operation") 
    498498        }; 
    499         int ret = JOptionPane.showOptionDialog( 
     499        int ret = OptionPaneUtil.showOptionDialog( 
    500500                Main.parent, 
    501501                tr("<html>There are {0} additional primitives referred to by relation {1}<br>" 
    … …  
    507507                        JOptionPane.YES_NO_OPTION, 
    508508                        JOptionPane.QUESTION_MESSAGE, 
    509                         null, 
    510509                        options, 
    511510                        options[0] 

  • trunk/src/org/openstreetmap/josm/gui/conflict/properties/PropertiesMerger.java

    r1690 r1847  
    2525import org.openstreetmap.josm.data.coor.LatLon; 
    2626import org.openstreetmap.josm.data.osm.OsmPrimitiveType; 
     27import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2728import org.openstreetmap.josm.gui.conflict.MergeDecisionType; 
    2829import org.openstreetmap.josm.tools.ImageProvider; 
    … …  
    582583                    tr("No, abort") 
    583584            }; 
    584             int ret = JOptionPane.showOptionDialog( 
     585            int ret = OptionPaneUtil.showOptionDialog( 
    585586                    null, 
    586587                    tr("<html>To keep your local version, JOSM<br>" 
    … …  
    595596                    JOptionPane.YES_NO_OPTION, 
    596597                    JOptionPane.QUESTION_MESSAGE, 
    597                     null, 
    598598                    options, 
    599599                    options[1] 
    … …  
    624624                    tr("No, abort") 
    625625            }; 
    626             int ret = JOptionPane.showOptionDialog( 
     626            int ret = OptionPaneUtil.showOptionDialog( 
    627627                    null, 
    628628                    tr("<html>JOSM will have to remove your local primitive with id {0}<br>" 
    … …  
    634634                    JOptionPane.YES_NO_OPTION, 
    635635                    JOptionPane.QUESTION_MESSAGE, 
    636                     null, 
    637636                    options, 
    638637                    options[1] 

  • trunk/src/org/openstreetmap/josm/gui/dialogs/ConflictResolutionDialog.java

    r1782 r1847  
    2222import org.openstreetmap.josm.Main; 
    2323import org.openstreetmap.josm.command.Command; 
     24import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2425import org.openstreetmap.josm.gui.conflict.ConflictResolver; 
    2526import org.openstreetmap.josm.gui.conflict.properties.OperationCancelledException; 
    … …  
    174175                        tr("Apply partial resolutions"), 
    175176                        tr("Continue resolving")}; 
    176                 int n = JOptionPane.showOptionDialog(null, 
     177                int n = OptionPaneUtil.showOptionDialog(null, 
    177178                        tr("<html>You didn''t finish to resolve all conflicts.<br>" 
    178179                                + "Click <strong>{0}</strong> to apply already resolved conflicts anyway.<br>" 
    … …  
    184185                        JOptionPane.YES_NO_OPTION, 
    185186                        JOptionPane.WARNING_MESSAGE, 
    186                         null, 
    187187                        options, 
    188188                        options[1] 

  • trunk/src/org/openstreetmap/josm/gui/dialogs/HistoryDialog.java

    r1815 r1847  
    4343import org.openstreetmap.josm.data.osm.history.History; 
    4444import org.openstreetmap.josm.data.osm.history.HistoryDataSet; 
     45import org.openstreetmap.josm.gui.OptionPaneUtil; 
    4546import org.openstreetmap.josm.gui.PleaseWaitRunnable; 
    4647import org.openstreetmap.josm.gui.SideButton; 
    … …  
    246247                msg = task.getLastException().toString(); 
    247248            } 
    248             JOptionPane.showMessageDialog( 
     249            OptionPaneUtil.showMessageDialog( 
    249250                    Main.parent, 
    250251                    tr( 

  • trunk/src/org/openstreetmap/josm/gui/dialogs/LayerListPopup.java

    r1169 r1847  
    1313 
    1414import org.openstreetmap.josm.Main; 
     15import org.openstreetmap.josm.gui.OptionPaneUtil; 
    1516import org.openstreetmap.josm.gui.layer.Layer; 
    1617import org.openstreetmap.josm.tools.ImageProvider; 
    … …  
    2829        } 
    2930        public void actionPerformed(ActionEvent e) { 
    30             JOptionPane.showMessageDialog(Main.parent, layer.getInfoComponent()); 
     31            OptionPaneUtil.showMessageDialog( 
     32                    Main.parent, 
     33                    layer.getInfoComponent(), 
     34                    tr("Information about layer"), 
     35                    JOptionPane.INFORMATION_MESSAGE 
     36            ); 
    3137        } 
    3238    } 
    3339 
    3440    public LayerListPopup(final JList layers, final Layer layer) { 
    35         for (Component c : layer.getMenuEntries()) 
     41        for (Component c : layer.getMenuEntries()) { 
    3642            add(c); 
     43        } 
    3744    } 
    3845} 

  • trunk/src/org/openstreetmap/josm/gui/dialogs/RelationListDialog.java

    r1838 r1847  
    1616import javax.swing.DefaultListModel; 
    1717import javax.swing.JList; 
    18 import javax.swing.JOptionPane; 
    1918import javax.swing.JPanel; 
    2019import javax.swing.JScrollPane; 
    … …  
    2827import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    2928import org.openstreetmap.josm.data.osm.Relation; 
    30 import org.openstreetmap.josm.gui.ConditionalOptionPaneUtil; 
    31 import org.openstreetmap.josm.gui.OptionPaneUtil; 
    3229import org.openstreetmap.josm.gui.OsmPrimitivRenderer; 
    3330import org.openstreetmap.josm.gui.SideButton; 

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/ChildRelationBrowser.java

    r1828 r1847  
    3434import org.openstreetmap.josm.data.osm.RelationMember; 
    3535import org.openstreetmap.josm.data.osm.visitor.MergeVisitor; 
     36import org.openstreetmap.josm.gui.OptionPaneUtil; 
    3637import org.openstreetmap.josm.gui.PleaseWaitRunnable; 
    3738import org.openstreetmap.josm.gui.PrimitiveNameFormatter; 
    … …  
    309310                msg = lastException.toString(); 
    310311            } 
    311             JOptionPane.showMessageDialog(null, msg, tr("Error"), JOptionPane.ERROR_MESSAGE); 
     312            OptionPaneUtil.showMessageDialog( 
     313                    Main.parent, 
     314                    msg, 
     315                    tr("Error"), 
     316                    JOptionPane.ERROR_MESSAGE 
     317            ); 
    312318        } 
    313319 
    … …  
    331337 
    332338            if (conflictsCount > 0) { 
    333                 JOptionPane op = new JOptionPane( 
     339                OptionPaneUtil.showMessageDialog( 
     340                        Main.parent, 
    334341                        tr("There were {0} conflicts during import.", conflictsCount), 
     342                        tr("Conflicts in data"), 
    335343                        JOptionPane.WARNING_MESSAGE 
    336344                ); 
    337                 JDialog dialog = op.createDialog(ChildRelationBrowser.this, tr("Conflicts in data")); 
    338                 dialog.setAlwaysOnTop(true); 
    339                 dialog.setModal(true); 
    340                 dialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE); 
    341                 dialog.setVisible(true); 
    342345            } 
    343346        } 
    … …  
    423426                msg = lastException.toString(); 
    424427            } 
    425             JOptionPane.showMessageDialog(null, msg, tr("Error"), JOptionPane.ERROR_MESSAGE); 
     428            OptionPaneUtil.showMessageDialog( 
     429                    Main.parent, 
     430                    msg, 
     431                    tr("Error"), 
     432                    JOptionPane.ERROR_MESSAGE 
     433            ); 
    426434        } 
    427435 
    … …  
    445453 
    446454            if (conflictsCount > 0) { 
    447                 JOptionPane op = new JOptionPane( 
     455                OptionPaneUtil.showMessageDialog( 
     456                        Main.parent, 
    448457                        tr("There were {0} conflicts during import.", conflictsCount), 
     458                        tr("Conflicts in data"), 
    449459                        JOptionPane.WARNING_MESSAGE 
    450460                ); 
    451                 JDialog dialog = op.createDialog(ChildRelationBrowser.this, tr("Conflicts in data")); 
    452                 dialog.setAlwaysOnTop(true); 
    453                 dialog.setModal(true); 
    454                 dialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE); 
    455                 dialog.setVisible(true); 
    456461            } 
    457462        } 

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/GenericRelationEditor.java

    r1845 r1847  
    713713                    this.nameFormatter.getName(primitive) 
    714714            ); 
    715             OptionPaneUtil.showMessageDialog(Main.parent, msg, tr("Warning"), JOptionPane.WARNING_MESSAGE); 
     715            OptionPaneUtil.showMessageDialog( 
     716                    Main.parent, 
     717                    msg, 
     718                    tr("Warning"), 
     719                    JOptionPane.WARNING_MESSAGE 
     720            ); 
    716721        } 
    717722 
    … …  
    986991                Conflict<Relation> conflict = new Conflict<Relation>(getRelation(), editedRelation); 
    987992                getLayer().getConflicts().add(conflict); 
    988                 JOptionPane op = new JOptionPane( 
     993                OptionPaneUtil.showMessageDialog( 
     994                        Main.parent, 
    989995                        tr("<html>The relation has changed outside of the editor.<br>" 
    990996                                + "Your edits can't be applied directly, a conflict has been created instead.</html>"), 
     997                                tr("Warning"), 
    991998                                JOptionPane.WARNING_MESSAGE 
    992999                ); 
    993                 JDialog dialog = op.createDialog(GenericRelationEditor.this, tr("Conflict created")); 
    994                 dialog.setAlwaysOnTop(true); 
    995                 dialog.setModal(true); 
    996                 dialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE); 
    997                 dialog.setVisible(true); 
    9981000            } else { 
    9991001                tagEditorModel.applyToPrimitive(editedRelation); 
    … …  
    12921294                msg = lastException.toString(); 
    12931295            } 
    1294             JOptionPane.showMessageDialog(null, msg, tr("Error"), JOptionPane.ERROR_MESSAGE); 
     1296            OptionPaneUtil.showMessageDialog( 
     1297                    Main.parent, 
     1298                    msg, 
     1299                    tr("Error"), 
     1300                    JOptionPane.ERROR_MESSAGE 
     1301            ); 
    12951302        } 
    12961303 
    … …  
    13051312 
    13061313            if (conflictsCount > 0) { 
    1307                 JOptionPane op = new JOptionPane(tr("There were {0} conflicts during import.", conflictsCount), 
    1308                         JOptionPane.WARNING_MESSAGE); 
    1309                 JDialog dialog = op.createDialog(GenericRelationEditor.this, tr("Conflicts in data")); 
    1310                 dialog.setAlwaysOnTop(true); 
    1311                 dialog.setModal(true); 
    1312                 dialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE); 
    1313                 dialog.setVisible(true); 
     1314                OptionPaneUtil.showMessageDialog( 
     1315                        Main.parent, 
     1316                        tr("There were {0} conflicts during import.", conflictsCount), 
     1317                        tr("Warning"), 
     1318                        JOptionPane.WARNING_MESSAGE 
     1319                ); 
    13141320            } 
    13151321        } 

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/ReferringRelationsBrowser.java

    r1836 r1847  
    3232import org.openstreetmap.josm.data.osm.Relation; 
    3333import org.openstreetmap.josm.data.osm.visitor.MergeVisitor; 
     34import org.openstreetmap.josm.gui.OptionPaneUtil; 
    3435import org.openstreetmap.josm.gui.OsmPrimitivRenderer; 
    3536import org.openstreetmap.josm.gui.PleaseWaitRunnable; 
    … …  
    202203                msg = lastException.toString(); 
    203204            } 
    204             JOptionPane.showMessageDialog( 
    205                     null, 
     205            OptionPaneUtil.showMessageDialog( 
     206                    Main.parent, 
    206207                    msg, 
    207208                    tr("Error"), 
    … …  
    259260                        return; 
    260261                    getLayer().getConflicts().add(visitor.getConflicts()); 
    261                     JOptionPane op = new JOptionPane( 
     262                    OptionPaneUtil.showMessageDialog( 
     263                            Main.parent, 
    262264                            tr("There were {0} conflicts during import.", 
    263265                                    visitor.getConflicts().size()), 
     266                                    tr("Warning"), 
    264267                                    JOptionPane.WARNING_MESSAGE 
    265268                    ); 
    266                     JDialog dialog = op.createDialog(ReferringRelationsBrowser.this, tr("Conflicts in data")); 
    267                     dialog.setAlwaysOnTop(true); 
    268                     dialog.setModal(true); 
    269                     dialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE); 
    270                     dialog.setVisible(true); 
    271269                } 
    272270            } catch(Exception e) { 

  • trunk/src/org/openstreetmap/josm/gui/download/BookmarkSelection.java

    r1169 r1847  
    2323import org.openstreetmap.josm.data.Preferences; 
    2424import org.openstreetmap.josm.gui.BookmarkList; 
     25import org.openstreetmap.josm.gui.OptionPaneUtil; 
    2526import org.openstreetmap.josm.tools.GBC; 
    2627 
    … …  
    7879 
    7980                if (tempBookmark == null) { 
    80                     JOptionPane.showMessageDialog(Main.parent, tr("Please enter the desired coordinates first.")); 
     81                    OptionPaneUtil.showMessageDialog( 
     82                            Main.parent, 
     83                            tr("Please enter the desired coordinates first."), 
     84                            tr("Information"), 
     85                            JOptionPane.INFORMATION_MESSAGE 
     86                    ); 
    8187                    return; 
    8288                } 
    … …  
    94100                Object sel = bookmarks.getSelectedValue(); 
    95101                if (sel == null) { 
    96                     JOptionPane.showMessageDialog(Main.parent,tr("Select a bookmark first.")); 
     102                    OptionPaneUtil.showMessageDialog( 
     103                            Main.parent, 
     104                            tr("Select a bookmark first."), 
     105                            tr("Information"), 
     106                            JOptionPane.INFORMATION_MESSAGE 
     107                    ); 
    97108                    return; 
    98109                } 

  • trunk/src/org/openstreetmap/josm/gui/download/PlaceSelection.java

    r1349 r1847  
    3232 
    3333import org.openstreetmap.josm.Main; 
     34import org.openstreetmap.josm.gui.OptionPaneUtil; 
    3435import org.openstreetmap.josm.gui.download.DownloadDialog; 
    3536import org.openstreetmap.josm.gui.download.DownloadSelection; 
    … …  
    166167                    if(searchtext.length()==0) 
    167168                    { 
    168                         JOptionPane.showMessageDialog(Main.parent,tr("Please enter a search string")); 
     169                        OptionPaneUtil.showMessageDialog( 
     170                                Main.parent, 
     171                                tr("Please enter a search string"), 
     172                                tr("Information"), 
     173                                JOptionPane.INFORMATION_MESSAGE 
     174                        ); 
    169175                    } 
    170176                    else 
    … …  
    173179                        component.repaint(); 
    174180                        URL url = new URL("http://gazetteer.openstreetmap.org/namefinder/search.xml?find=" 
    175                         +java.net.URLEncoder.encode(searchTerm.getText(), "UTF-8")); 
     181                                +java.net.URLEncoder.encode(searchTerm.getText(), "UTF-8")); 
    176182                        HttpURLConnection activeConnection = (HttpURLConnection)url.openConnection(); 
    177183                        //System.out.println("got return: "+activeConnection.getResponseCode()); 
    … …  
    318324    // if bounding box selected on other tab, de-select item 
    319325    public void boundingBoxChanged(DownloadDialog gui) { 
    320         if (!updatingSelf) searchResultDisplay.clearSelection(); 
     326        if (!updatingSelf) { 
     327            searchResultDisplay.clearSelection(); 
     328        } 
    321329    } 
    322330} 

  • trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/ImageMarker.java

    r1169 r1847  
    6161            public void actionPerformed(ActionEvent ev) { 
    6262                p.setCursor(Cursor.getPredefinedCursor(Cursor.WAIT_CURSOR)); 
    63                 if (scale.getModel().isSelected()) 
     63                if (scale.getModel().isSelected()) { 
    6464                    ((JLabel)vp.getView()).setIcon(loadScaledImage(imageUrl, Math.max(vp.getWidth(), vp.getHeight()))); 
    65                 else 
     65                } else { 
    6666                    ((JLabel)vp.getView()).setIcon(new ImageIcon(imageUrl)); 
     67                } 
    6768                p.setCursor(Cursor.getDefaultCursor()); 
    6869            } 
    … …  
    7273        JDialog dlg = pane.createDialog(Main.parent, imageUrl.toString()); 
    7374        dlg.setModal(false); 
     75        dlg.toFront(); 
    7476        dlg.setVisible(true); 
    7577    } 
Note: See TracChangeset for help on using the changeset viewer.