Changeset 3894 in josm for trunk/src/org/openstreetmap


Ignore:
Timestamp:
2011-02-13T10:30:28+01:00 (14 years ago)
Author:
bastiK
Message:

integrate wireframe into mappaint dialog; rename 'shortdescription' to 'title' (xml header key is unchanged)

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

Legend:

Unmodified
Added
Removed

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

    r3083 r3894  
    6868        setEnabled(Main.map != null && Main.main.getEditLayer() != null);
    6969    }
     70
     71    public boolean isSelected() {
     72        return selected;
     73    }
    7074}

  • trunk/src/org/openstreetmap/josm/gui/MainMenu.java

    r3837 r3894  
    138138
    139139    /* View menu */
     140    public final WireframeToggleAction wireFrameToggleAction = new WireframeToggleAction();
    140141    public final JosmAction toggleGPXLines = new ToggleGPXLinesAction();
    141142    public final InfoAction info = new InfoAction();
    … …  
    272273
    273274        // -- wireframe toggle action
    274         WireframeToggleAction wireFrameToggleAction = new WireframeToggleAction();
    275275        final JCheckBoxMenuItem wireframe = new JCheckBoxMenuItem(wireFrameToggleAction);
    276276        viewMenu.add(wireframe);

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

    r3882 r3894  
    99import java.awt.Font;
    1010import java.awt.GridBagLayout;
     11import java.awt.Insets;
    1112import java.awt.Point;
    1213import java.awt.Rectangle;
    1314import java.awt.event.ActionEvent;
     15import java.awt.event.ActionListener;
    1416import java.awt.event.KeyEvent;
    1517import java.awt.event.MouseEvent;
    … …  
    2628
    2729import javax.swing.AbstractAction;
     30import javax.swing.DefaultButtonModel;
    2831import javax.swing.DefaultListSelectionModel;
     32import javax.swing.JCheckBox;
    2933import javax.swing.JFileChooser;
    3034import javax.swing.JLabel;
    … …  
    3842import javax.swing.ListSelectionModel;
    3943import javax.swing.SingleSelectionModel;
     44import javax.swing.SwingConstants;
    4045import javax.swing.SwingUtilities;
    4146import javax.swing.UIManager;
     47import javax.swing.border.EmptyBorder;
    4248import javax.swing.event.ChangeEvent;
    4349import javax.swing.event.ChangeListener;
    … …  
    4652import javax.swing.table.AbstractTableModel;
    4753import javax.swing.table.DefaultTableCellRenderer;
     54import javax.swing.table.TableCellRenderer;
    4855import javax.swing.table.TableModel;
    4956
    … …  
    7279    protected OnOffAction onoffAction;
    7380    protected ReloadAction reloadAction;
     81    protected MoveUpDownAction upAction;
     82    protected MoveUpDownAction downAction;
     83    protected JCheckBox cbWireframe;
    7484
    7585    public MapPaintDialog() {
    … …  
    8494
    8595        model = new StylesModel();
     96
     97        cbWireframe = new JCheckBox();
     98        JLabel wfLabel = new JLabel(tr("Wireframe View"), ImageProvider.get("dialogs/mappaint", "wireframe_small"), JLabel.HORIZONTAL);
     99        wfLabel.setFont(wfLabel.getFont().deriveFont(Font.PLAIN));
     100
     101        cbWireframe.setModel(new DefaultButtonModel() {
     102            @Override
     103            public void setSelected(boolean b) {
     104                super.setSelected(b);
     105                tblStyles.setEnabled(!b);
     106                onoffAction.updateEnabledState();
     107                upAction.updateEnabledState();
     108                downAction.updateEnabledState();
     109            }
     110        });
     111        cbWireframe.addActionListener(new ActionListener() {
     112            public void actionPerformed(ActionEvent e) {
     113                Main.main.menu.wireFrameToggleAction.actionPerformed(null);
     114            }
     115        });
     116        cbWireframe.setBorder(new EmptyBorder(new Insets(1,1,1,1)));
    86117       
    87118        tblStyles = new StylesTable(model);
    … …  
    94125        tblStyles.getColumnModel().getColumn(0).setMaxWidth(1);
    95126        tblStyles.getColumnModel().getColumn(0).setResizable(false);
     127        tblStyles.getColumnModel().getColumn(0).setCellRenderer(new MyCheckBoxRenderer());
    96128        tblStyles.getColumnModel().getColumn(1).setCellRenderer(new StyleSourceRenderer());
    97129        tblStyles.setShowGrid(false);
    98130        tblStyles.setIntercellSpacing(new Dimension(0, 0));
    99131
    100         pnl.add(new JScrollPane(tblStyles), BorderLayout.CENTER);
     132        JPanel p = new JPanel(new GridBagLayout());
     133        p.add(cbWireframe, GBC.std(0, 0));
     134        p.add(wfLabel, GBC.std(1, 0).weight(1, 0));
     135        p.add(tblStyles, GBC.std(0, 1).span(2).fill());
     136
     137        pnl.add(new JScrollPane(p), BorderLayout.CENTER);
    101138
    102139        pnl.add(buildButtonRow(), BorderLayout.SOUTH);
    … …  
    126163        reloadAction = new ReloadAction();
    127164        onoffAction = new OnOffAction();
    128         MoveUpDownAction up = new MoveUpDownAction(false);
    129         MoveUpDownAction down = new MoveUpDownAction(true);
     165        upAction = new MoveUpDownAction(false);
     166        downAction = new MoveUpDownAction(true);
    130167        selectionModel.addListSelectionListener(onoffAction);
    131168        selectionModel.addListSelectionListener(reloadAction);
    132         selectionModel.addListSelectionListener(up);
    133         selectionModel.addListSelectionListener(down);
     169        selectionModel.addListSelectionListener(upAction);
     170        selectionModel.addListSelectionListener(downAction);
    134171        p.add(new SideButton(onoffAction));
    135         p.add(new SideButton(up));
    136         p.add(new SideButton(down));
     172        p.add(new SideButton(upAction));
     173        p.add(new SideButton(downAction));
    137174        p.add(new SideButton(new LaunchMapPaintPreferencesAction()));
    138175
    … …  
    143180    public void showNotify() {
    144181        MapPaintStyles.addMapPaintSylesUpdateListener(model);
     182        Main.main.menu.wireFrameToggleAction.addButtonModel(cbWireframe.getModel());
    145183    }
    146184
    147185    @Override
    148186    public void hideNotify() {
     187        Main.main.menu.wireFrameToggleAction.removeButtonModel(cbWireframe.getModel());
    149188        MapPaintStyles.removeMapPaintSylesUpdateListener(model);
    150189    }
    … …  
    232271    }
    233272
    234     private static class StyleSourceRenderer extends DefaultTableCellRenderer {
     273    private class MyCheckBoxRenderer extends JCheckBox implements TableCellRenderer {
     274
     275        public MyCheckBoxRenderer() {
     276            setHorizontalAlignment(SwingConstants.CENTER);
     277            setVerticalAlignment(SwingConstants.CENTER);
     278        }
     279
     280        public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus,int row,int column) {
     281            boolean b = (Boolean) value;
     282            setSelected(b);
     283            setEnabled(!cbWireframe.isSelected());
     284            return this;
     285        }
     286    }
     287
     288    private class StyleSourceRenderer extends DefaultTableCellRenderer {
    235289        @Override
    236290        public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) {
    … …  
    240294            label.setIcon(s.getIcon());
    241295            label.setToolTipText(s.getToolTipText());
     296            label.setEnabled(!cbWireframe.isSelected());
    242297            return label;
    243298        }
    … …  
    252307
    253308        protected void updateEnabledState() {
    254             setEnabled(tblStyles.getSelectedRowCount() > 0);
     309            setEnabled(!cbWireframe.isSelected() && tblStyles.getSelectedRowCount() > 0);
    255310        }
    256311
    … …  
    274329     * The action to move down the currently selected entries in the list.
    275330     */
    276     class MoveUpDownAction extends AbstractAction implements ListSelectionListener {
     331    protected class MoveUpDownAction extends AbstractAction implements ListSelectionListener {
    277332
    278333        final int increment;
    … …  
    287342        public void updateEnabledState() {
    288343            int[] sel = tblStyles.getSelectedRows();
    289             setEnabled(MapPaintStyles.canMoveStyles(sel, increment));
     344            setEnabled(!cbWireframe.isSelected() && MapPaintStyles.canMoveStyles(sel, increment));
    290345        }
    291346
    … …  
    328383            putValue(SHORT_DESCRIPTION, tr("reload selected styles from file"));
    329384            putValue(SMALL_ICON, ImageProvider.get("dialogs", "refresh"));
    330             updateEnabledState();
    331         }
    332 
    333         protected void updateEnabledState() {
     385            setEnabled(getEnabledState());
     386        }
     387
     388        protected boolean getEnabledState() {
     389            if (cbWireframe.isSelected())
     390                return false;
    334391            int[] pos = tblStyles.getSelectedRows();
    335             boolean e = pos.length > 0;
     392            if (pos.length == 0)
     393                return false;
    336394            for (int i : pos) {
    337                 if (!model.getRow(i).isLocal()) {
    338                     e = false;
    339                     break;
    340                 }
    341             }
    342             setEnabled(e);
     395                if (!model.getRow(i).isLocal())
     396                    return false;
     397            }
     398            return true;
    343399        }
    344400
    345401        @Override
    346402        public void valueChanged(ListSelectionEvent e) {
    347             updateEnabledState();
     403            setEnabled(getEnabledState());
    348404        }
    349405
    … …  
    599655        @Override
    600656        public void launch(MouseEvent evt) {
     657            if (cbWireframe.isSelected())
     658                return;
    601659            Point p = evt.getPoint();
    602660            int index = tblStyles.rowAtPoint(p);

  • trunk/src/org/openstreetmap/josm/gui/mappaint/StyleSource.java

    r3876 r3894  
    1515
    1616import org.openstreetmap.josm.data.osm.OsmPrimitive;
     17import org.openstreetmap.josm.gui.mappaint.MapPaintStyles.IconReference;
    1718import org.openstreetmap.josm.gui.preferences.SourceEntry;
    1819import org.openstreetmap.josm.tools.ImageProvider;
    … …  
    2324    public File zipIcons;
    2425
    25     public StyleSource(String url, String name, String shortdescription) {
    26         super(url, name, shortdescription, true);
     26    private ImageIcon imageIcon;
     27
     28    /******
     29     * The following fields is additional information found in the header
     30     * of the source file.
     31     */
     32   
     33    public String icon;
     34
     35    public StyleSource(String url, String name, String title) {
     36        super(url, name, title, true);
    2737    }
    2838
    2939    public StyleSource(SourceEntry entry) {
    30         super(entry.url, entry.name, entry.shortdescription, entry.active);
     40        super(entry.url, entry.name, entry.title, entry.active);
    3141    }
    3242
    … …  
    4555    }
    4656
    47     protected void clearErrors() {
     57    protected void init() {
    4858        errors.clear();
     59        imageIcon = null;
     60        icon = null;
    4961    }
    5062
    51     private static ImageIcon pencil;
     63    private static ImageIcon defaultIcon;
     64
     65    private static ImageIcon getDefaultIcon() {
     66        if (defaultIcon == null) {
     67            defaultIcon = ImageProvider.get("dialogs/mappaint", "pencil");
     68        }
     69        return defaultIcon;
     70    }
    5271
    5372    protected ImageIcon getSourceIcon() {
    54         if (pencil == null) {
    55             pencil = ImageProvider.get("dialogs/mappaint", "pencil");
     73        if (imageIcon == null) {
     74            if (icon != null) {
     75                imageIcon = MapPaintStyles.getIcon(new IconReference(icon, this), false);
     76            }
     77            if (imageIcon == null) {
     78                imageIcon = getDefaultIcon();
     79            }
    5680        }
    57         return pencil;
     81        return imageIcon;
    5882    }
    5983

  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleSource.java

    r3893 r3894  
    4141    @Override
    4242    public void loadStyleSource() {
     43        init();
    4344        rules.clear();
    44         clearErrors();
    4545        try {
    4646            MapCSSParser parser = new MapCSSParser(getSourceInputStream(), "UTF-8");
    … …  
    100100        }
    101101        Cascade c = mc.getCascade("default");
    102         String sd = c.get("title", null, String.class);
    103         if (sd != null) {
    104             this.shortdescription = sd;
     102        String pTitle = c.get("title", null, String.class);
     103        if (title == null) {
     104            title = pTitle;
     105        }
     106        String pIcon = c.get("icon", null, String.class);
     107        if (icon == null) {
     108            icon = pIcon;
    105109        }
    106110    }

  • trunk/src/org/openstreetmap/josm/gui/mappaint/xml/XmlStyleSource.java

    r3893 r3894  
    5050
    5151    protected void init() {
    52         clearErrors();
     52        super.init();
    5353        icons.clear();
    5454        lines.clear();

  • trunk/src/org/openstreetmap/josm/gui/mappaint/xml/XmlStyleSourceHandler.java

    r3863 r3894  
    160160                    style.name = atts.getValue("name");
    161161                }
    162                 if (style.shortdescription == null) {
    163                     style.shortdescription = atts.getValue("shortdescription");
     162                if (style.title == null) {
     163                    style.title = atts.getValue("shortdescription");
     164                }
     165                if (style.icon == null) {
     166                    style.icon = atts.getValue("icon");
    164167                }
    165168            }

  • trunk/src/org/openstreetmap/josm/gui/preferences/MapPaintPreference.java

    r3870 r3894  
    242242            ExtendedSourceEntry i = new ExtendedSourceEntry("elemstyles.xml", "resource://styles/standard/elemstyles.xml");
    243243            i.name = "standard";
    244             i.shortdescription = tr("Internal Style");
     244            i.title = tr("JOSM Internal Style");
    245245            i.description = tr("Internal style to be used as base for runtime switchable overlay styles");
    246246            return Collections.singletonList(i);
    … …  
    249249        @Override
    250250        public Collection<String> serialize(SourceEntry entry) {
    251             return Arrays.asList(new String[] {entry.url, entry.name, entry.shortdescription, Boolean.toString(entry.active)});
     251            return Arrays.asList(new String[] {entry.url, entry.name, entry.title, Boolean.toString(entry.active)});
    252252        }
    253253

  • trunk/src/org/openstreetmap/josm/gui/preferences/SourceEditor.java

    r3855 r3894  
    621621            super(url, null, null, true);
    622622            this.simpleFileName = simpleFileName;
    623             version = author = link = description = shortdescription = null;
     623            version = author = link = description = title = null;
    624624        }
    625625
    … …  
    628628         */
    629629        public String getDisplayName() {
    630             return shortdescription == null ? simpleFileName : shortdescription;
     630            return title == null ? simpleFileName : title;
    631631        }
    632632
    … …  
    656656    protected class EditSourceEntryDialog extends ExtendedDialog {
    657657
    658         /**
    659          * We call this text field "name", but it is actually the shortdescription.
    660          */
    661         private JTextField tfName;
     658        private JTextField tfTitle;
    662659        private JTextField tfURL;
    663660        private JCheckBox cbActive;
    … …  
    670667            JPanel p = new JPanel(new GridBagLayout());
    671668
    672             tfName = new JTextField(60);
     669            tfTitle = new JTextField(60);
    673670            p.add(new JLabel(tr("Name (optional):")), GBC.std().insets(15, 0, 5, 5));
    674             p.add(tfName, GBC.eol().insets(0, 0, 5, 5));
     671            p.add(tfTitle, GBC.eol().insets(0, 0, 5, 5));
    675672
    676673            tfURL = new JTextField(60);
    … …  
    682679
    683680            if (e != null) {
    684                 if (e.shortdescription != null) {
    685                     tfName.setText(e.shortdescription);
     681                if (e.title != null) {
     682                    tfTitle.setText(e.title);
    686683                }
    687684                tfURL.setText(e.url);
    … …  
    738735        }
    739736
    740         public String getShortdescription() {
    741             return tfName.getText();
     737        public String getTitle() {
     738            return tfTitle.getText();
    742739        }
    743740
    … …  
    773770                activeSourcesModel.addSource(new SourceEntry(
    774771                        editEntryDialog.getURL(),
    775                         null, editEntryDialog.getShortdescription(), active));
     772                        null, editEntryDialog.getTitle(), active));
    776773                activeSourcesModel.fireTableDataChanged();
    777774            }
    … …  
    828825            editEntryDialog.showDialog();
    829826            if (editEntryDialog.getValue() == 1) {
    830                 if (e.shortdescription != null || !equal(editEntryDialog.getShortdescription(), "")) {
    831                     e.shortdescription = editEntryDialog.getShortdescription();
    832                     if (equal(e.shortdescription, "")) {
    833                         e.shortdescription = null;
     827                if (e.title != null || !equal(editEntryDialog.getTitle(), "")) {
     828                    e.title = editEntryDialog.getTitle();
     829                    if (equal(e.title, "")) {
     830                        e.title = null;
    834831                    }
    835832                }
    … …  
    11741171                            } else if ("description".equals(key) && last.description == null) {
    11751172                                last.description = value;
    1176                             } else if ("shortdescription".equals(key) && last.shortdescription == null) {
    1177                                 last.shortdescription = value;
     1173                            } else if ((lang + "shortdescription").equals(key) && last.title == null) {
     1174                                last.title = value;
     1175                            } else if ("shortdescription".equals(key) && last.title == null) {
     1176                                last.title = value;
    11781177                            } else if ("name".equals(key) && last.name == null) {
    11791178                                last.name = value;
    … …  
    11841183                            } else if ((lang + "description").equals(key)) {
    11851184                                last.description = value;
    1186                             } else if ((lang + "shortdescription").equals(key)) {
    1187                                 last.shortdescription = value;
    11881185                            }
    11891186                        }
    … …  
    12221219        private String fromSourceEntry(SourceEntry entry) {
    12231220            StringBuilder s = new StringBuilder("<html><b>");
    1224             if (entry.shortdescription != null) {
    1225                 s.append(entry.shortdescription).append("</b> (");
     1221            if (entry.title != null) {
     1222                s.append(entry.title).append("</b> (");
    12261223            }
    12271224            s.append(entry.url);
    1228             if (entry.shortdescription != null) {
     1225            if (entry.title != null) {
    12291226                s.append(")");
    12301227            }

  • trunk/src/org/openstreetmap/josm/gui/preferences/SourceEntry.java

    r3886 r3894  
    3232
    3333    /**
    34      * A short description that can be used as menu entry.
     34     * A title that can be used as menu entry.
    3535     */
    36     public String shortdescription;
     36    public String title;
    3737
    3838    /**
    … …  
    4545        this.url = url;
    4646        this.name = equal(name, "") ? null : name;
    47         this.shortdescription = equal(shortdescription, "") ? null : shortdescription;
     47        this.title = equal(shortdescription, "") ? null : shortdescription;
    4848        this.active = active;
    4949    }
    … …  
    5252        this.url = e.url;
    5353        this.name = e.name;
    54         this.shortdescription = e.shortdescription;
     54        this.title = e.title;
    5555        this.active = e.active;
    5656    }
    … …  
    6363        return equal(other.url, url) &&
    6464                equal(other.name, name) &&
    65                 equal(other.shortdescription, shortdescription) &&
     65                equal(other.title, title) &&
    6666                other.active == active;
    6767    }
    … …  
    7272        hash = 89 * hash + (this.url != null ? this.url.hashCode() : 0);
    7373        hash = 89 * hash + (this.name != null ? this.name.hashCode() : 0);
    74         hash = 89 * hash + (this.shortdescription != null ? this.shortdescription.hashCode() : 0);
     74        hash = 89 * hash + (this.title != null ? this.title.hashCode() : 0);
    7575        hash = 89 * hash + (this.active ? 1 : 0);
    7676        return hash;
    … …  
    7979    @Override
    8080    public String toString() {
    81         return shortdescription != null ? shortdescription : url;
     81        return title != null ? title : url;
    8282    }
    8383
    … …  
    8888     */
    8989    public String getDisplayString() {
    90         if (shortdescription != null)
    91             return shortdescription;
     90        if (title != null)
     91            return title;
    9292        else
    9393            return getFileNamePart();

  • trunk/src/org/openstreetmap/josm/gui/preferences/TaggingPresetPreference.java

    r3885 r3894  
    329329        public Collection<ExtendedSourceEntry> getDefault() {
    330330            ExtendedSourceEntry i = new ExtendedSourceEntry("defaultpresets.xml", "resource://data/defaultpresets.xml");
    331             i.shortdescription = tr("Internal Preset");
     331            i.title = tr("Internal Preset");
    332332            i.description = tr("The default preset for JOSM");
    333333            return Collections.singletonList(i);
    … …  
    336336        @Override
    337337        public Collection<String> serialize(SourceEntry entry) {
    338             return Arrays.asList(new String[] {entry.url, entry.shortdescription});
     338            return Arrays.asList(new String[] {entry.url, entry.title});
    339339        }
    340340

  • trunk/src/org/openstreetmap/josm/tools/GBC.java

    r3338 r3894  
    120120        return new Box.Filler(new Dimension(x,y), new Dimension(x,y), new Dimension(maxx,maxy));
    121121    }
     122
     123    public GBC grid(int x, int y) {
     124        gridx = x;
     125        gridy = y;
     126        return this;
     127    }
     128
     129    public GBC span(int width, int height) {
     130        gridwidth = width;
     131        gridheight = height;
     132        return this;
     133    }
     134
     135    public GBC span(int width) {
     136        gridwidth = width;
     137        return this;
     138    }
     139
     140    public static GBC std(int x, int y) {
     141        GBC c = new GBC();
     142        c.anchor = WEST;
     143        c.gridx = x;
     144        c.gridy = y;
     145        return c;
     146    }
     147
    122148}
Note: See TracChangeset for help on using the changeset viewer.