Ignore:
Timestamp:
2017-10-12T16:02:15+02:00 (7 years ago)
Author:
bastiK
Message:

see #15410 - change preferences scheme for named colors - makes runtime color name registry obsolete

Location:
trunk/src/org/openstreetmap/josm/gui/layer
Files:
5 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java

    r12671 r12987  
    2626import org.openstreetmap.josm.data.gpx.GpxTrack;
    2727import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
    28 import org.openstreetmap.josm.data.preferences.ColorProperty;
     28import org.openstreetmap.josm.data.preferences.NamedColorProperty;
    2929import org.openstreetmap.josm.data.projection.Projection;
    3030import org.openstreetmap.josm.gui.MapView;
     
    100100
    101101    @Override
    102     protected ColorProperty getBaseColorProperty() {
     102    protected NamedColorProperty getBaseColorProperty() {
    103103        return GpxDrawHelper.DEFAULT_COLOR;
    104104    }
  • trunk/src/org/openstreetmap/josm/gui/layer/Layer.java

    r12782 r12987  
    2929import org.openstreetmap.josm.data.preferences.AbstractProperty;
    3030import org.openstreetmap.josm.data.preferences.AbstractProperty.ValueChangeListener;
    31 import org.openstreetmap.josm.data.preferences.ColorProperty;
     31import org.openstreetmap.josm.data.preferences.NamedColorProperty;
    3232import org.openstreetmap.josm.data.projection.Projection;
    3333import org.openstreetmap.josm.data.projection.ProjectionChangeListener;
     
    204204     */
    205205    public AbstractProperty<Color> getColorProperty() {
    206         ColorProperty base = getBaseColorProperty();
     206        NamedColorProperty base = getBaseColorProperty();
    207207        if (base != null) {
    208             // cannot cache this - name may change.
    209             return base.getChildColor("layer " + getName());
     208            return base.getChildColor(NamedColorProperty.COLOR_CATEGORY_LAYER, getName(), base.getName());
    210209        } else {
    211210            return null;
     
    218217     * @since 10824
    219218     */
    220     protected ColorProperty getBaseColorProperty() {
     219    protected NamedColorProperty getBaseColorProperty() {
    221220        return null;
    222221    }
  • trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

    r12846 r12987  
    8181import org.openstreetmap.josm.data.osm.visitor.paint.Rendering;
    8282import org.openstreetmap.josm.data.osm.visitor.paint.relations.MultipolygonCache;
    83 import org.openstreetmap.josm.data.preferences.ColorProperty;
    8483import org.openstreetmap.josm.data.preferences.IntegerProperty;
     84import org.openstreetmap.josm.data.preferences.NamedColorProperty;
    8585import org.openstreetmap.josm.data.preferences.StringProperty;
    8686import org.openstreetmap.josm.data.projection.Projection;
     
    153153    public static final StringProperty PROPERTY_SAVE_EXTENSION = new StringProperty("save.extension.osm", "osm");
    154154
    155     private static final ColorProperty PROPERTY_BACKGROUND_COLOR = new ColorProperty(marktr("background"), Color.BLACK);
    156     private static final ColorProperty PROPERTY_OUTSIDE_COLOR = new ColorProperty(marktr("outside downloaded area"), Color.YELLOW);
     155    private static final NamedColorProperty PROPERTY_BACKGROUND_COLOR = new NamedColorProperty(marktr("background"), Color.BLACK);
     156    private static final NamedColorProperty PROPERTY_OUTSIDE_COLOR = new NamedColorProperty(marktr("outside downloaded area"), Color.YELLOW);
    157157
    158158    /** List of recent relations */
  • trunk/src/org/openstreetmap/josm/gui/layer/gpx/GpxDrawHelper.java

    r12891 r12987  
    4343import org.openstreetmap.josm.data.gpx.GpxData.GpxDataChangeListener;
    4444import org.openstreetmap.josm.data.gpx.WayPoint;
    45 import org.openstreetmap.josm.data.preferences.ColorProperty;
     45import org.openstreetmap.josm.data.preferences.NamedColorProperty;
    4646import org.openstreetmap.josm.gui.MapView;
    4747import org.openstreetmap.josm.gui.MapViewState;
     
    6969     * @since 10824
    7070     */
    71     public static final ColorProperty DEFAULT_COLOR = new ColorProperty(marktr("gps point"), Color.magenta);
     71    public static final NamedColorProperty DEFAULT_COLOR = new NamedColorProperty(marktr("gps point"), Color.magenta);
    7272
    7373    private final GpxData data;
     
    266266    public Color getColor(String layerName, boolean ignoreCustom) {
    267267        if (ignoreCustom || getColorMode(layerName) == ColorMode.NONE) {
    268             return DEFAULT_COLOR.getChildColor(specName(layerName)).get();
     268            return DEFAULT_COLOR.getChildColor(
     269                    NamedColorProperty.COLOR_CATEGORY_LAYER,
     270                    layerName,
     271                    DEFAULT_COLOR.getName()).get();
    269272        } else {
    270273            return null;
  • trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/MarkerLayer.java

    r12846 r12987  
    3838import org.openstreetmap.josm.data.gpx.WayPoint;
    3939import org.openstreetmap.josm.data.osm.visitor.BoundingXYVisitor;
    40 import org.openstreetmap.josm.data.preferences.ColorProperty;
     40import org.openstreetmap.josm.data.preferences.NamedColorProperty;
    4141import org.openstreetmap.josm.gui.MainApplication;
    4242import org.openstreetmap.josm.gui.MapView;
     
    7979
    8080    private static final Color DEFAULT_COLOR = Color.magenta;
    81     private static final ColorProperty COLOR_PROPERTY = new ColorProperty(marktr("gps marker"), DEFAULT_COLOR);
     81    private static final NamedColorProperty COLOR_PROPERTY = new NamedColorProperty(marktr("gps marker"), DEFAULT_COLOR);
    8282
    8383    /**
     
    163163
    164164    @Override
    165     protected ColorProperty getBaseColorProperty() {
     165    protected NamedColorProperty getBaseColorProperty() {
    166166        return COLOR_PROPERTY;
    167167    }
Note: See TracChangeset for help on using the changeset viewer.