Changeset 2331 in josm


Ignore:
Timestamp:
2009-10-27T14:51:46+01:00 (15 years ago)
Author:
Gubaer
Message:

Cleanup in DownloadAction and DownloadDialog
Added context-sensitive help to DownloadDialog

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

Legend:

Unmodified
Added
Removed

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

    r2330 r2331  
    3535public class DownloadAction extends JosmAction {
    3636    private static final Logger logger = Logger.getLogger(DownloadAction.class.getName());
    37    
    38     private DownloadDialog dialog;
    39     private ExtendedDialog downloadDialog;
    40 
     37 
    4138    public DownloadAction() {
    4239        super(tr("Download from OSM..."), "download", tr("Download map data from the OSM server."),
    … …  
    4542    }
    4643
    47     /**
    48      * Creates the download dialog
    49      *
    50      * @return the downlaod dialog
    51      */
    52     protected ExtendedDialog createDownloadDialog() {
    53         if (dialog == null)
    54             dialog = new DownloadDialog();
    55         dialog.restoreSettings();
    56 
    57         final String prefName = dialog.getClass().getName()+ ".geometry";
    58         final WindowGeometry wg = WindowGeometry.centerInWindow(Main.parent,
    59                 new Dimension(1000,600));
    60 
    61         if (downloadDialog == null) {
    62             JPanel downPanel = new JPanel(new BorderLayout());
    63             downPanel.add(dialog, BorderLayout.CENTER);
    64             downloadDialog= new ExtendedDialog(Main.parent,
    65                 tr("Download"),
    66                 new String[] {tr("OK"), tr("Cancel")});
    67             downloadDialog.setContent(downPanel, false /* don't use a scroll pane inside the dialog */);
    68             downloadDialog.setButtonIcons(new String[] {"ok", "cancel"});
    69             downloadDialog.setRememberWindowGeometry(prefName, wg);
    70         }
    71         return downloadDialog;
    72     }
    73 
    7444    public void actionPerformed(ActionEvent e) {
    75         ExtendedDialog dlg = createDownloadDialog();
    76         boolean finish = false;
    77         while (!finish) {           
    78             dlg.showDialog();
    79             if (dlg.getValue() == 1 /* OK */) {
    80                 dialog.rememberSettings();
    81                 Bounds area = dialog.getSelectedDownloadArea();               
    82                 if (dialog.isDownloadOsmData()) {
    83                     DownloadOsmTask task = new DownloadOsmTask();
    84                     Future<?> future = task.download(dialog.isNewLayerRequired(), area, null);
    85                     Main.worker.submit(new PostDownloadHandler(task, future));
    86                     finish = true;
    87                 }
    88                 if (dialog.isDownloadGpxData()) {
    89                     DownloadGpsTask task = new DownloadGpsTask();
    90                     Future<?> future = task.download(dialog.isNewLayerRequired(),area, null);
    91                     Main.worker.submit(new PostDownloadHandler(task, future));
    92                     finish = true;
    93                 }
    94             } else {
    95                 finish = true;
     45        DownloadDialog dialog = DownloadDialog.getInstance();
     46        dialog.setVisible(true);
     47        if (! dialog.isCanceled()) {
     48            dialog.rememberSettings();
     49            Bounds area = dialog.getSelectedDownloadArea();               
     50            if (dialog.isDownloadOsmData()) {
     51                DownloadOsmTask task = new DownloadOsmTask();
     52                Future<?> future = task.download(dialog.isNewLayerRequired(), area, null);
     53                Main.worker.submit(new PostDownloadHandler(task, future));
    9654            }
    97             if (!finish) {
    98                 JOptionPane.showMessageDialog(
    99                         Main.parent,
    100                         tr("Please select at least one task to download"),
    101                         tr("Error"),
    102                         JOptionPane.ERROR_MESSAGE
    103                 );
     55            if (dialog.isDownloadGpxData()) {
     56                DownloadGpsTask task = new DownloadGpsTask();
     57                Future<?> future = task.download(dialog.isNewLayerRequired(),area, null);
     58                Main.worker.submit(new PostDownloadHandler(task, future));
    10459            }
    105         }
    106     }
     60        } 
     61    }   
    10762}

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

    r2330 r2331  
    44import static org.openstreetmap.josm.tools.I18n.tr;
    55
     6import java.awt.BorderLayout;
    67import java.awt.Color;
     8import java.awt.Component;
     9import java.awt.Dimension;
     10import java.awt.FlowLayout;
    711import java.awt.Font;
    812import java.awt.GridBagLayout;
    … …  
    1317import java.awt.event.InputEvent;
    1418import java.awt.event.KeyEvent;
     19import java.awt.event.WindowAdapter;
     20import java.awt.event.WindowEvent;
    1521import java.util.ArrayList;
    1622import java.util.List;
    … …  
    1925import javax.swing.AbstractAction;
    2026import javax.swing.JCheckBox;
     27import javax.swing.JComponent;
     28import javax.swing.JDialog;
    2129import javax.swing.JLabel;
     30import javax.swing.JOptionPane;
    2231import javax.swing.JPanel;
    2332import javax.swing.JTabbedPane;
    … …  
    2736import org.openstreetmap.josm.data.Bounds;
    2837import org.openstreetmap.josm.gui.MapView;
     38import org.openstreetmap.josm.gui.SideButton;
     39import org.openstreetmap.josm.gui.help.ContextSensitiveHelpAction;
     40import org.openstreetmap.josm.gui.help.HelpUtil;
    2941import org.openstreetmap.josm.plugins.PluginHandler;
    3042import org.openstreetmap.josm.tools.GBC;
     43import org.openstreetmap.josm.tools.ImageProvider;
    3144import org.openstreetmap.josm.tools.OsmUrlToBounds;
     45import org.openstreetmap.josm.tools.WindowGeometry;
     46import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
    3247
    3348/**
    34  * Main download dialog.
    35  *
    36  * Can be extended by plugins in two ways:
    37  * (1) by adding download tasks that are then called with the selected bounding box
    38  * (2) by adding "DownloadSelection" objects that implement different ways of selecting a bounding box
    39  *
    40  * @author Frederik Ramm <frederik@remote.org>
    41  *
     49 *
    4250 */
    43 public class DownloadDialog extends JPanel {
     51public class DownloadDialog extends JDialog {
    4452    static private final Logger logger = Logger.getLogger(DownloadDialog.class.getName());
     53   
     54    /** the unique instance of the download dialog */
     55    static private DownloadDialog instance;
     56   
     57    /**
     58     * Replies the unique instance of the download dialog
     59     *
     60     * @return the unique instance of the download dialog
     61     */
     62    static public DownloadDialog getInstance() {
     63        if (instance == null)
     64            instance = new DownloadDialog(Main.parent);
     65        return instance;
     66    }   
    4567
    4668    private final List<DownloadSelection> downloadSelections = new ArrayList<DownloadSelection>();
    4769    private final JTabbedPane tpDownloadAreaSelectors = new JTabbedPane();
    48     private final JCheckBox cbNewLayer;
     70    private JCheckBox cbNewLayer;
    4971    private final JLabel sizeCheck = new JLabel();
    50 
    5172    private Bounds currentBounds = null;
     73    private boolean canceled;
    5274
    5375    private JCheckBox cbDownloadOsmData = new JCheckBox(tr("OpenStreetMap data"), true);
    5476    private JCheckBox cbDownloadGpxData = new JCheckBox(tr("Raw GPS data"));
    5577
    56 
    57     public DownloadDialog() {
    58         setLayout(new GridBagLayout());
     78   
     79    public JPanel buildMainPanel() {
     80        JPanel pnl = new JPanel();
     81        pnl.setLayout(new GridBagLayout());
    5982
    6083        // adding the download tasks
    61         add(new JLabel(tr("Data Sources and Types")), GBC.eol().insets(0,5,0,0));
    62         add(cbDownloadOsmData,  GBC.eol().insets(20,0,0,0));
    63         add(cbDownloadGpxData,  GBC.eol().insets(20,0,0,0));
     84        pnl.add(new JLabel(tr("Data Sources and Types")), GBC.eol().insets(0,5,0,0));
     85        pnl.add(cbDownloadOsmData,  GBC.eol().insets(20,0,0,0));
     86        pnl.add(cbDownloadGpxData,  GBC.eol().insets(20,0,0,0));
    6487       
    6588        // predefined download selections
    … …  
    81104   
    82105        cbNewLayer = new JCheckBox(tr("Download as new layer"));
    83         add(cbNewLayer, GBC.eol().insets(0,5,0,0));
    84 
    85         add(new JLabel(tr("Download Area")), GBC.eol().insets(0,5,0,0));
    86         add(tpDownloadAreaSelectors, GBC.eol().fill());
     106        pnl.add(cbNewLayer, GBC.eol().insets(0,5,0,0));
     107
     108        pnl. add(new JLabel(tr("Download Area")), GBC.eol().insets(0,5,0,0));
     109        pnl.add(tpDownloadAreaSelectors, GBC.eol().fill());
    87110
    88111        try {
    … …  
    94117        Font labelFont = sizeCheck.getFont();
    95118        sizeCheck.setFont(labelFont.deriveFont(Font.PLAIN, labelFont.getSize()));
    96         add(sizeCheck, GBC.eop().insets(0,5,5,10));
    97 
    98         getInputMap(WHEN_IN_FOCUSED_WINDOW).put(
     119        pnl.add(sizeCheck, GBC.eop().insets(0,5,5,10));
     120        return pnl;
     121    }
     122   
     123    protected JPanel buildButtonPanel() {
     124        JPanel pnl = new JPanel();
     125        pnl.setLayout(new FlowLayout());
     126       
     127        pnl.add(new SideButton(new DownloadAction()));
     128        pnl.add(new SideButton(new CancelAction()));
     129        pnl.add(new SideButton(new ContextSensitiveHelpAction(ht("/Dialog/DownloadDialog"))));
     130        return pnl;       
     131    }
     132   
     133    public DownloadDialog(Component parent) {
     134        super(JOptionPane.getFrameForComponent(parent),tr("Download"), true /* modal */);
     135        getContentPane().setLayout(new BorderLayout());
     136        getContentPane().add(buildMainPanel(), BorderLayout.CENTER);
     137        getContentPane().add(buildButtonPanel(), BorderLayout.SOUTH);
     138       
     139        getRootPane().getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW).put(
    99140                KeyStroke.getKeyStroke(KeyEvent.VK_V, InputEvent.CTRL_MASK), "checkClipboardContents");
    100141
    101         getActionMap().put("checkClipboardContents", new AbstractAction() {
     142        getRootPane().getActionMap().put("checkClipboardContents", new AbstractAction() {
    102143            public void actionPerformed(ActionEvent e) {
    103144                checkClipboardContents();
    104145            }
    105146        });
    106        
     147        HelpUtil.setHelpContext(getRootPane(), ht("/Dialog/DownloadDialog"));
     148        addWindowListener(new WindowEventHandler());
    107149        restoreSettings();
    108150    }
     151   
    109152
    110153    private void checkClipboardContents() {
    … …  
    247290    }
    248291   
     292    @Override
     293    public void setVisible(boolean visible) {
     294        if (visible) {
     295            new WindowGeometry(
     296                    getClass().getName() + ".geometry",
     297                    WindowGeometry.centerInWindow(
     298                            getParent(),
     299                            new Dimension(1000,600)
     300                    )
     301            ).apply(this);
     302        } else if (!visible && isShowing()){
     303            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
     304        }
     305        super.setVisible(visible);
     306    }
     307
     308    /**
     309     * Replies true if the dialog was canceled
     310     *
     311     * @return true if the dialog was canceled
     312     */
     313    public boolean isCanceled() {
     314        return canceled;
     315    }
     316
     317    protected void setCanceled(boolean canceled) {
     318        this.canceled = canceled;
     319    }
     320   
     321    class CancelAction extends AbstractAction {
     322        public CancelAction() {
     323            putValue(NAME, tr("Cancel"));
     324            putValue(SMALL_ICON, ImageProvider.get("cancel"));
     325            putValue(SHORT_DESCRIPTION, tr("Click to close the dialog and to abort downloading"));           
     326        }
     327       
     328        public void run() {
     329            setCanceled(true);
     330            setVisible(false);   
     331        }
     332       
     333        public void actionPerformed(ActionEvent e) {
     334            run();
     335        }       
     336    }
     337
     338    class DownloadAction extends AbstractAction {
     339        public DownloadAction() {
     340            putValue(NAME, tr("Download"));
     341            putValue(SMALL_ICON, ImageProvider.get("download"));
     342            putValue(SHORT_DESCRIPTION, tr("Click do download the currently selected area"));           
     343        }
     344       
     345        public void actionPerformed(ActionEvent e) {
     346            if (currentBounds == null) {
     347                JOptionPane.showMessageDialog(
     348                        DownloadDialog.this,
     349                        tr("Please select a download area first."),
     350                        tr("Error"),
     351                        JOptionPane.ERROR_MESSAGE
     352                );
     353                return;
     354            }
     355            if (!isDownloadOsmData() && !isDownloadOsmData()) {
     356                JOptionPane.showMessageDialog(
     357                        DownloadDialog.this,
     358                        tr("<html>Neither <strong>{0}</strong> nor <strong>{1}</strong> is enabled.<br>"
     359                                + "Please chose to either download OSM data, or GPX data, or both.</html>",
     360                        cbDownloadOsmData.getText(),
     361                        cbDownloadGpxData.getText()
     362                        ),
     363                        tr("Error"),
     364                        JOptionPane.ERROR_MESSAGE
     365                );
     366                return;
     367            }
     368            setCanceled(false);
     369            setVisible(false);           
     370        }       
     371    }
     372   
     373    class WindowEventHandler extends WindowAdapter {
     374        @Override
     375        public void windowClosing(WindowEvent e) {
     376            new CancelAction().run();
     377        }       
     378    }
    249379}

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

    r2327 r2331  
    233233        submitSearch.addActionListener(new ActionListener() {
    234234            public void actionPerformed(ActionEvent e) {
    235                 queryServer(gui);
     235                queryServer(gui.getRootPane());
    236236            }
    237237        });
    … …  
    239239        searchTerm.addActionListener(new ActionListener() {
    240240            public void actionPerformed(ActionEvent e) {
    241                 queryServer(gui);
     241                queryServer(gui.getRootPane());
    242242            }
    243243        });
Note: See TracChangeset for help on using the changeset viewer.