Ignore:
Timestamp:
2012-02-11T18:56:53+01:00 (12 years ago)
Author:
simon04
Message:

fix #7370 - Refactor Command.getDescription

Location:
trunk/src/org/openstreetmap/josm/command
Files:
24 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/command/AddCommand.java

    r3262 r4918  
    88import java.util.Collections;
    99
    10 import javax.swing.JLabel;
     10import javax.swing.Icon;
    1111
    1212import org.openstreetmap.josm.data.osm.OsmPrimitive;
    … …  
    6060    }
    6161
    62     @Override public JLabel getDescription() {
     62    @Override
     63    public String getDescriptionText() {
    6364        String msg;
    6465        switch(OsmPrimitiveType.from(osm)) {
    … …  
    6869        default: /* should not happen */msg = ""; break;
    6970        }
     71        return tr(msg, osm.getDisplayName(DefaultNameFormatter.getInstance()));
     72    }
    7073
    71         return new JLabel(
    72                 tr(msg, osm.getDisplayName(DefaultNameFormatter.getInstance())),
    73                 ImageProvider.get(OsmPrimitiveType.from(osm)),
    74                 JLabel.HORIZONTAL);
     74    @Override
     75    public Icon getDescriptionIcon() {
     76        return ImageProvider.get(OsmPrimitiveType.from(osm));
    7577    }
    7678

  • trunk/src/org/openstreetmap/josm/command/AddPrimitivesCommand.java

    r3703 r4918  
    88import java.util.HashSet;
    99import java.util.List;
     10import javax.swing.Icon;
    1011
    1112import javax.swing.JLabel;
    … …  
    101102    }
    102103
    103     @Override public JLabel getDescription() {
     104    @Override
     105    public String getDescriptionText() {
    104106        int size = data != null ? data.size() : createdPrimitives.size();
    105         return new JLabel(trn("Added {0} object", "Added {0} objects", size, size), null,
    106                 JLabel.HORIZONTAL
    107         );
     107        return trn("Added {0} object", "Added {0} objects", size, size);
     108    }
     109
     110    @Override
     111    public Icon getDescriptionIcon() {
     112        return null;
    108113    }
    109114

  • trunk/src/org/openstreetmap/josm/command/ChangeCommand.java

    r3262 r4918  
    66
    77import java.util.Collection;
    8 
    9 import javax.swing.JLabel;
     8import javax.swing.Icon;
    109
    1110import org.openstreetmap.josm.data.osm.OsmPrimitive;
    … …  
    5049    }
    5150
    52     @Override public JLabel getDescription() {
     51    @Override
     52    public String getDescriptionText() {
    5353        String msg = "";
    5454        switch(OsmPrimitiveType.from(osm)) {
    … …  
    5757        case RELATION: msg = marktr("Change relation {0}"); break;
    5858        }
    59         return new JLabel(tr(msg,
    60                     osm.getDisplayName(DefaultNameFormatter.getInstance())),
    61                     ImageProvider.get(OsmPrimitiveType.from(osm)),
    62                     JLabel.HORIZONTAL);
     59        return tr(msg, osm.getDisplayName(DefaultNameFormatter.getInstance()));
     60    }
     61
     62    @Override
     63    public Icon getDescriptionIcon() {
     64        return ImageProvider.get(OsmPrimitiveType.from(osm));
    6365    }
    6466}

  • trunk/src/org/openstreetmap/josm/command/ChangeNodesCommand.java

    r3262 r4918  
    66import java.util.Collection;
    77import java.util.List;
    8 
    9 import javax.swing.JLabel;
     8import javax.swing.Icon;
    109
    1110import org.openstreetmap.josm.data.osm.Node;
    … …  
    4645    }
    4746
    48     @Override public JLabel getDescription() {
    49         String msg = tr("Changed nodes of {0}", way.getDisplayName(DefaultNameFormatter.getInstance()));
    50         return new JLabel(msg,
    51                         ImageProvider.get(OsmPrimitiveType.WAY),
    52                         JLabel.HORIZONTAL);
     47    @Override
     48    public String getDescriptionText() {
     49        return tr("Changed nodes of {0}", way.getDisplayName(DefaultNameFormatter.getInstance()));
     50    }
     51
     52    @Override
     53    public Icon getDescriptionIcon() {
     54        return ImageProvider.get(OsmPrimitiveType.WAY);
    5355    }
    5456}

  • trunk/src/org/openstreetmap/josm/command/ChangePropertyCommand.java

    r4773 r4918  
    1414import java.util.List;
    1515import java.util.Map;
    16 
    17 import javax.swing.JLabel;
     16import javax.swing.Icon;
    1817
    1918import org.openstreetmap.josm.Main;
    … …  
    145144    }
    146145
    147     @Override public JLabel getDescription() {
     146    @Override
     147    public String getDescriptionText() {
    148148        String text;
    149149        if (objects.size() == 1 && tags.size() == 1) {
    … …  
    187187                text = tr("Set {0} properties for {1} objects", tags.size(), objects.size());
    188188        }
    189         return new JLabel(text, ImageProvider.get("data", "key"), JLabel.HORIZONTAL);
     189        return text;
     190    }
     191
     192    @Override
     193    public Icon getDescriptionIcon() {
     194        return ImageProvider.get("data", "key");
    190195    }
    191196
    … …  
    196201        for (final OsmPrimitive osm : objects) {
    197202            children.add(new PseudoCommand() {
    198                 @Override public JLabel getDescription() {
    199                     return new JLabel(
    200                                 osm.getDisplayName(DefaultNameFormatter.getInstance()),
    201                                 ImageProvider.get(OsmPrimitiveType.from(osm)),
    202                                 JLabel.HORIZONTAL);
    203 
    204                 }
     203                @Override public String getDescriptionText() {
     204                    return osm.getDisplayName(DefaultNameFormatter.getInstance());
     205                }
     206
     207                @Override public Icon getDescriptionIcon() {
     208                    return ImageProvider.get(OsmPrimitiveType.from(osm));
     209                }
     210
    205211                @Override public Collection<? extends OsmPrimitive> getParticipatingPrimitives() {
    206212                    return Collections.singleton(osm);

  • trunk/src/org/openstreetmap/josm/command/ChangePropertyKeyCommand.java

    r4806 r4918  
    1010import java.util.LinkedList;
    1111import java.util.List;
     12import javax.swing.Icon;
    1213
    13 import javax.swing.JLabel;
    14 
    15 import org.openstreetmap.josm.command.Command;
    16 import org.openstreetmap.josm.command.PseudoCommand;
    1714import org.openstreetmap.josm.data.osm.OsmPrimitive;
    1815import org.openstreetmap.josm.data.validation.util.NameVisitor;
    … …  
    7168
    7269    @Override
    73     public JLabel getDescription() {
     70    public String getDescriptionText() {
    7471        String text = tr( "Replace \"{0}\" by \"{1}\" for", key, newKey);
    7572        if (objects.size() == 1) {
    … …  
    8077            text += " "+objects.size()+" "+trn("object","objects",objects.size());
    8178        }
    82         return new JLabel(text, ImageProvider.get("data", "key"), JLabel.HORIZONTAL);
     79        return text;
     80    }
     81
     82    @Override
     83    public Icon getDescriptionIcon() {
     84        return ImageProvider.get("data", "key");
    8385    }
    8486
    … …  
    9496            children.add(new PseudoCommand() {
    9597                @Override
    96                 public JLabel getDescription() {
    97                     return v.toLabel();
     98                public String getDescriptionText() {
     99                    return v.name;
     100                }
     101                @Override
     102                public Icon getDescriptionIcon() {
     103                    return v.icon;
    98104                }
    99105                @Override

  • trunk/src/org/openstreetmap/josm/command/ChangeRelationMemberRoleCommand.java

    r4191 r4918  
    66import java.util.Collection;
    77
    8 import javax.swing.JLabel;
    9 
    10 import org.openstreetmap.josm.Main;
     8import javax.swing.Icon;
    119import org.openstreetmap.josm.data.osm.OsmPrimitive;
    1210import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    … …  
    6462    }
    6563
    66     @Override public JLabel getDescription() {
    67         return new JLabel(
    68                         tr("Change relation member role for {0} {1}",
    69                                 OsmPrimitiveType.from(relation),
    70                                 relation.getDisplayName(DefaultNameFormatter.getInstance())
    71                         ),
    72                         ImageProvider.get(OsmPrimitiveType.from(relation)),
    73                         JLabel.HORIZONTAL
    74         );
     64    @Override
     65    public String getDescriptionText() {
     66        return tr("Change relation member role for {0} {1}",
     67                OsmPrimitiveType.from(relation),
     68                relation.getDisplayName(DefaultNameFormatter.getInstance()));
     69    }
     70
     71    @Override
     72    public Icon getDescriptionIcon() {
     73        return ImageProvider.get(OsmPrimitiveType.from(relation));
    7574    }
    7675}

  • trunk/src/org/openstreetmap/josm/command/Command.java

    r4908 r4918  
    160160    @Override public Collection<? extends OsmPrimitive> getParticipatingPrimitives() {
    161161        return cloneMap.keySet();
    162     }
    163 
    164     public String getDescrpitionText() {
    165         Object o = getDescription();
    166         if (o instanceof JLabel) {
    167             return ((JLabel) o).getText();
    168         } else {
    169             return o.toString();
    170         }
    171162    }
    172163

  • trunk/src/org/openstreetmap/josm/command/ConflictAddCommand.java

    r3995 r4918  
    55
    66import java.util.Collection;
    7 
    8 import javax.swing.JLabel;
     7import javax.swing.Icon;
    98import javax.swing.JOptionPane;
    109
    … …  
    6362    }
    6463
    65     @Override public JLabel getDescription() {
    66         return new JLabel(
    67                 tr("Add conflict for ''{0}''",
    68                         conflict.getMy().getDisplayName(DefaultNameFormatter.getInstance())
    69                 ),
    70                 ImageProvider.get(OsmPrimitiveType.from(conflict.getMy())),
    71                 JLabel.HORIZONTAL
    72         );
     64    @Override
     65    public String getDescriptionText() {
     66        return tr("Add conflict for ''{0}''",
     67                conflict.getMy().getDisplayName(DefaultNameFormatter.getInstance()));
     68    }
     69
     70    @Override
     71    public Icon getDescriptionIcon() {
     72        return ImageProvider.get(OsmPrimitiveType.from(conflict.getMy()));
    7373    }
    7474}

  • trunk/src/org/openstreetmap/josm/command/CoordinateConflictResolveCommand.java

    r3262 r4918  
    55
    66import java.util.Collection;
    7 
    8 import javax.swing.JLabel;
     7import javax.swing.Icon;
    98
    109import org.openstreetmap.josm.data.conflict.Conflict;
    … …  
    3837    }
    3938
    40     @Override public JLabel getDescription() {
    41         return new JLabel(
    42                         tr("Resolve conflicts in coordinates in {0}",conflict.getMy().getId()),
    43                         ImageProvider.get("data", "object"),
    44                         JLabel.HORIZONTAL
    45         );
     39    @Override
     40    public String getDescriptionText() {
     41        return tr("Resolve conflicts in coordinates in {0}", conflict.getMy().getId());
     42    }
     43
     44    @Override
     45    public Icon getDescriptionIcon() {
     46        return ImageProvider.get("data", "object");
    4647    }
    4748

  • trunk/src/org/openstreetmap/josm/command/DeleteCommand.java

    r4677 r4918  
    1919import java.util.Set;
    2020import java.util.Map.Entry;
     21import javax.swing.Icon;
    2122
    2223import javax.swing.JLabel;
    … …  
    149150    }
    150151
    151     @Override public JLabel getDescription() {
    152         if (toDelete.size() == 1) {
     152    private Set<OsmPrimitiveType> getTypesToDelete() {
     153        Set<OsmPrimitiveType> typesToDelete = new HashSet<OsmPrimitiveType>();
     154        for (OsmPrimitive osm : toDelete) {
     155            typesToDelete.add(OsmPrimitiveType.from(osm));
     156        }
     157        return typesToDelete;
     158    }
     159
     160    @Override
     161    public String getDescriptionText() {
     162           if (toDelete.size() == 1) {
    153163            OsmPrimitive primitive = toDelete.iterator().next();
    154164            String msg = "";
    … …  
    159169            }
    160170
    161             return new JLabel(tr(msg, primitive.getDisplayName(DefaultNameFormatter.getInstance())),
    162                     ImageProvider.get(OsmPrimitiveType.from(primitive)), JLabel.HORIZONTAL);
     171            return tr(msg, primitive.getDisplayName(DefaultNameFormatter.getInstance()));
    163172        } else {
    164             Set<OsmPrimitiveType> typesToDelete = new HashSet<OsmPrimitiveType>();
    165             for (OsmPrimitive osm : toDelete) {
    166                 typesToDelete.add(OsmPrimitiveType.from(osm));
    167             }
     173            Set<OsmPrimitiveType> typesToDelete = getTypesToDelete();
    168174            String msg = "";
    169             String apiname = "object";
    170175            if (typesToDelete.size() > 1) {
    171176                msg = trn("Delete {0} object", "Delete {0} objects", toDelete.size(), toDelete.size());
    172177            } else {
    173178                OsmPrimitiveType t = typesToDelete.iterator().next();
    174                 apiname = t.getAPIName();
    175179                switch(t) {
    176180                case NODE: msg = trn("Delete {0} node", "Delete {0} nodes", toDelete.size(), toDelete.size()); break;
    … …  
    179183                }
    180184            }
    181             return  new JLabel(msg, ImageProvider.get("data", apiname), JLabel.HORIZONTAL);
     185            return msg;
     186        }
     187    }
     188
     189    @Override
     190    public Icon getDescriptionIcon() {
     191        if (toDelete.size() == 1) {
     192            return ImageProvider.get(OsmPrimitiveType.from(toDelete.iterator().next()));
     193        }
     194        Set<OsmPrimitiveType> typesToDelete = getTypesToDelete();
     195        if (typesToDelete.size() > 1) {
     196            return ImageProvider.get("data", "object");
     197        } else {
     198            return ImageProvider.get(typesToDelete.iterator().next());
    182199        }
    183200    }
    … …  
    190207            for (final OsmPrimitive osm : toDelete) {
    191208                children.add(new PseudoCommand() {
    192                     @Override public JLabel getDescription() {
    193                         return new JLabel(
    194                                 tr("Deleted ''{0}''",
    195                                         osm.getDisplayName(DefaultNameFormatter.getInstance())),
    196                                         ImageProvider.get(OsmPrimitiveType.from(osm)), JLabel.HORIZONTAL);
     209
     210                    @Override public String getDescriptionText() {
     211                        return tr("Deleted ''{0}''", osm.getDisplayName(DefaultNameFormatter.getInstance()));
    197212                    }
     213
     214                    @Override public Icon getDescriptionIcon() {
     215                        return ImageProvider.get(OsmPrimitiveType.from(osm));
     216                    }
     217
    198218                    @Override public Collection<? extends OsmPrimitive> getParticipatingPrimitives() {
    199219                        return Collections.singleton(osm);

  • trunk/src/org/openstreetmap/josm/command/DeletedStateConflictResolveCommand.java

    r3262 r4918  
    66import java.util.Collection;
    77
    8 import javax.swing.JLabel;
     8import javax.swing.Icon;
    99
    1010import org.openstreetmap.josm.data.conflict.Conflict;
     11import org.openstreetmap.josm.data.osm.Node;
    1112import org.openstreetmap.josm.data.osm.OsmPrimitive;
    1213import org.openstreetmap.josm.gui.conflict.pair.MergeDecisionType;
    … …  
    3839    }
    3940
    40     @Override public JLabel getDescription() {
    41         return new JLabel(
    42                         tr("Resolve conflicts in deleted state in {0}",conflict.getMy().getId()),
    43                         ImageProvider.get("data", "object"),
    44                         JLabel.HORIZONTAL
    45         );
     41    @Override
     42    public String getDescriptionText() {
     43        return tr("Resolve conflicts in deleted state in {0}", conflict.getMy().getId());
     44    }
     45
     46    @Override
     47    public Icon getDescriptionIcon() {
     48        return ImageProvider.get("data", "object");
    4649    }
    4750

  • trunk/src/org/openstreetmap/josm/command/ModifiedConflictResolveCommand.java

    r3262 r4918  
    66
    77import java.util.Collection;
     8import javax.swing.Icon;
    89
    910import javax.swing.JLabel;
    … …  
    3334    }
    3435
    35     @Override public JLabel getDescription() {
     36    @Override
     37    public String getDescriptionText() {
    3638        String msg = "";
    3739        switch(OsmPrimitiveType.from(conflict.getMy())) {
    … …  
    4042        case RELATION: msg = marktr("Set the ''modified'' flag for relation {0}"); break;
    4143        }
    42         return new JLabel(
    43                         tr(msg,conflict.getMy().getId()),
    44                         ImageProvider.get("data", "object"),
    45                         JLabel.HORIZONTAL
    46         );
     44        return tr(msg,conflict.getMy().getId());
     45    }
     46
     47    @Override
     48    public Icon getDescriptionIcon() {
     49        return ImageProvider.get("data", "object");
    4750    }
    4851

  • trunk/src/org/openstreetmap/josm/command/MoveCommand.java

    r4894 r4918  
    99import java.util.LinkedList;
    1010import java.util.List;
     11import javax.swing.Icon;
    1112
    1213import javax.swing.JLabel;
    … …  
    131132    }
    132133
    133     @Override public JLabel getDescription() {
    134         return new JLabel(trn("Move {0} node", "Move {0} nodes", nodes.size(), nodes.size()), ImageProvider.get("data", "node"), JLabel.HORIZONTAL);
     134    @Override
     135    public String getDescriptionText() {
     136        return trn("Move {0} node", "Move {0} nodes", nodes.size(), nodes.size());
     137    }
     138
     139    @Override
     140    public Icon getDescriptionIcon() {
     141        return ImageProvider.get("data", "node");
    135142    }
    136143

  • trunk/src/org/openstreetmap/josm/command/PseudoCommand.java

    r3479 r4918  
    33
    44import java.util.Collection;
     5import javax.swing.Icon;
     6import javax.swing.JLabel;
    57
    68import org.openstreetmap.josm.data.osm.OsmPrimitive;
    … …  
    1416    /**
    1517     * Provide a description that can be presented in a list or tree view.
     18     * @deprecated This abstract method is deprecated.
     19     * On 2012-03-01, this abstract method will be removed.
     20     * At the same moment, the methods {@link #getDescrpitionText()} and {@link #getDescrpitionIcon()}
     21     * will be made abstract.
     22     * For a smooth transition, replace {@link #getDescription()} by implementations of
     23     * {@link #getDescrpitionText()} and {@link #getDescrpitionIcon()} as early as possible.
     24     * {@link #getDescription()} is no longer abstract and can therefore be removed.
    1625     */
    17     abstract public Object getDescription();
     26    @Deprecated
     27    public Object getDescription() {
     28        return null;
     29    }
     30
     31    /**
     32     * Provides a description text representing this command.
     33     */
     34    public String getDescriptionText() {
     35        Object o = getDescription();
     36        if (o instanceof JLabel) {
     37            return ((JLabel) o).getText();
     38        } else {
     39            return o.toString();
     40        }
     41    }
     42
     43    /**
     44     * Provides a descriptive icon of this command.
     45     */
     46    public Icon getDescriptionIcon() {
     47        Object o = getDescription();
     48        if (o instanceof JLabel) {
     49            return ((JLabel) o).getIcon();
     50        } else {
     51            return null;
     52        }
     53    }
    1854
    1955    /**

  • trunk/src/org/openstreetmap/josm/command/PurgeCommand.java

    r3486 r4918  
    1313import java.util.Set;
    1414
    15 import javax.swing.JLabel;
     15import javax.swing.Icon;
    1616
    1717import org.openstreetmap.josm.data.osm.DataSet;
    … …  
    252252
    253253    @Override
    254     public Object getDescription() {
    255         return new JLabel(trn("Purged {0} object", "Purged {0} objects", toPurge.size(), toPurge.size()), ImageProvider.get("data", "purge"), JLabel.HORIZONTAL);
     254    public String getDescriptionText() {
     255        return trn("Purged {0} object", "Purged {0} objects", toPurge.size(), toPurge.size());
     256    }
     257
     258    @Override
     259    public Icon getDescriptionIcon() {
     260        return ImageProvider.get("data", "purge");
    256261    }
    257262

  • trunk/src/org/openstreetmap/josm/command/RelationMemberConflictResolverCommand.java

    r4191 r4918  
    77import java.util.List;
    88
    9 import javax.swing.JLabel;
     9import javax.swing.Icon;
    1010
    1111import org.openstreetmap.josm.Main;
    … …  
    4646    }
    4747
    48     @Override public JLabel getDescription() {
    49         return new JLabel(
    50                         tr("Resolve conflicts in member list of relation {0}", my.getId()),
    51                         ImageProvider.get("data", "object"),
    52                         JLabel.HORIZONTAL
    53         );
     48    @Override
     49    public String getDescriptionText() {
     50        return tr("Resolve conflicts in member list of relation {0}", my.getId());
     51    }
     52
     53    @Override
     54    public Icon getDescriptionIcon() {
     55        return ImageProvider.get("data", "object");
    5456    }
    5557

  • trunk/src/org/openstreetmap/josm/command/RotateCommand.java

    r3702 r4918  
    66import java.util.Collection;
    77
    8 import javax.swing.JLabel;
     8import javax.swing.Icon;
    99
    1010import org.openstreetmap.josm.data.coor.EastNorth;
    … …  
    9999
    100100    @Override
    101     public JLabel getDescription() {
    102         return new JLabel(trn("Rotate {0} node", "Rotate {0} nodes", nodes.size(), nodes.size()), ImageProvider.get("data", "node"), JLabel.HORIZONTAL);
     101    public String getDescriptionText() {
     102        return trn("Rotate {0} node", "Rotate {0} nodes", nodes.size(), nodes.size());
     103    }
     104
     105    @Override
     106    public Icon getDescriptionIcon() {
     107        return ImageProvider.get("data", "node");
    103108    }
    104109}

  • trunk/src/org/openstreetmap/josm/command/ScaleCommand.java

    r3702 r4918  
    66import java.util.Collection;
    77
    8 import javax.swing.JLabel;
     8import javax.swing.Icon;
    99
    1010import org.openstreetmap.josm.data.coor.EastNorth;
    … …  
    8181
    8282    @Override
    83     public JLabel getDescription() {
    84         return new JLabel(trn("Scale {0} node", "Scale {0} nodes", nodes.size(), nodes.size()), ImageProvider.get("data", "node"), JLabel.HORIZONTAL);
     83    public String getDescriptionText() {
     84        return trn("Scale {0} node", "Scale {0} nodes", nodes.size(), nodes.size());
    8585    }
    8686
     87    @Override
     88    public Icon getDescriptionIcon() {
     89        return ImageProvider.get("data", "node");
     90    }
    8791}

  • trunk/src/org/openstreetmap/josm/command/SequenceCommand.java

    r4191 r4918  
    88import java.util.HashSet;
    99import java.util.List;
     10import javax.swing.Icon;
    1011
    1112import javax.swing.JLabel;
    … …  
    8788    }
    8889
    89     @Override public JLabel getDescription() {
    90         return new JLabel(tr("Sequence")+": "+name, ImageProvider.get("data", "sequence"), JLabel.HORIZONTAL);
     90    @Override
     91    public String getDescriptionText() {
     92        return tr("Sequence: {0}", name);
     93    }
     94
     95    @Override
     96    public Icon getDescriptionIcon() {
     97        return ImageProvider.get("data", "sequence");
    9198    }
    9299

  • trunk/src/org/openstreetmap/josm/command/TagConflictResolveCommand.java

    r4191 r4918  
    77import java.util.Collection;
    88import java.util.List;
     9import javax.swing.Icon;
    910
    10 import javax.swing.JLabel;
    1111
    1212import org.openstreetmap.josm.data.conflict.Conflict;
    … …  
    5555    }
    5656
    57     @Override public JLabel getDescription() {
     57    @Override
     58    public String getDescriptionText() {
    5859        String msg = "";
    5960        switch(OsmPrimitiveType.from(conflict.getMy())) {
    … …  
    6263        case RELATION: msg = marktr("Resolve {0} tag conflicts in relation {1}"); break;
    6364        }
    64         return new JLabel(
    65                         tr(msg,getNumDecidedConflicts(), conflict.getMy().getId()),
    66                         ImageProvider.get("data", "object"),
    67                         JLabel.HORIZONTAL
    68         );
     65        return tr(msg,getNumDecidedConflicts(), conflict.getMy().getId());
     66    }
     67
     68    @Override
     69    public Icon getDescriptionIcon() {
     70        return ImageProvider.get("data", "object");
    6971    }
    7072

  • trunk/src/org/openstreetmap/josm/command/TransformNodesCommand.java

    r3702 r4918  
    99import java.util.Map;
    1010
    11 import javax.swing.JLabel;
     11import javax.swing.Icon;
    1212
    1313import org.openstreetmap.josm.data.coor.EastNorth;
    … …  
    123123
    124124    @Override
    125     public JLabel getDescription() {
    126         return new JLabel(trn("Transform {0} node", "Transform {0} nodes", nodes.size(), nodes.size()), ImageProvider.get("data", "node"), JLabel.HORIZONTAL);
     125    public String getDescriptionText() {
     126        return trn("Transform {0} node", "Transform {0} nodes", nodes.size(), nodes.size());
     127    }
     128
     129    @Override
     130    public Icon getDescriptionIcon() {
     131        return ImageProvider.get("data", "node");
    127132    }
    128133

  • trunk/src/org/openstreetmap/josm/command/VersionConflictResolveCommand.java

    r3362 r4918  
    66
    77import java.util.Collection;
    8 
    9 import javax.swing.JLabel;
     8import javax.swing.Icon;
    109
    1110import org.openstreetmap.josm.data.conflict.Conflict;
    … …  
    3332    }
    3433
    35     @Override public JLabel getDescription() {
     34    @Override
     35    public String getDescriptionText() {
    3636        String msg = "";
    3737        switch(OsmPrimitiveType.from(conflict.getMy())) {
    … …  
    4040        case RELATION: msg = marktr("Resolve version conflict for relation {0}"); break;
    4141        }
    42         return new JLabel(
    43                         tr(msg,conflict.getMy().getId()),
    44                         ImageProvider.get("data", "object"),
    45                         JLabel.HORIZONTAL
    46         );
     42        return tr(msg, conflict.getMy().getId());
     43    }
     44
     45    @Override
     46    public Icon getDescriptionIcon() {
     47        return ImageProvider.get("data", "object");
    4748    }
    4849

  • trunk/src/org/openstreetmap/josm/command/WayNodesConflictResolverCommand.java

    r4191 r4918  
    77import java.util.List;
    88
    9 import javax.swing.JLabel;
     9import javax.swing.Icon;
    1010
    1111import org.openstreetmap.josm.data.conflict.Conflict;
    … …  
    4040        this.mergedNodeList = mergedNodeList;
    4141    }
     42    @Override
     43    public String getDescriptionText() {
     44        return tr("Resolve conflicts in node list of way {0}", conflict.getMy().getId());
     45    }
    4246
    43     @Override public JLabel getDescription() {
    44         return new JLabel(
    45                         tr("Resolve conflicts in node list of way {0}", conflict.getMy().getId()),
    46                         ImageProvider.get("data", "object"),
    47                         JLabel.HORIZONTAL
    48         );
     47    @Override
     48    public Icon getDescriptionIcon() {
     49        return ImageProvider.get("data", "object");
    4950    }
    5051
Note: See TracChangeset for help on using the changeset viewer.