Ignore:
Timestamp:
2008-08-17T14:45:48+02:00 (17 years ago)
Author:
stoecker
Message:

cleanup color preferences a lot

Location:
trunk/src/org/openstreetmap/josm/gui
Files:
10 edited

Legend:

Unmodified
Added
Removed

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

    r627 r804  
    33package org.openstreetmap.josm.gui;
    44
     5import static org.openstreetmap.josm.tools.I18n.marktr;
     6
     7import java.awt.Color;
    58import java.awt.Graphics;
    69import java.awt.geom.Rectangle2D;
    … …  
    3235                String text = dist > 1000 ? (Math.round(dist/100)/10.0)+"km" : Math.round(dist*10)/10+"m";
    3336                Rectangle2D bound = g.getFontMetrics().getStringBounds(text, g);
    34                 g.setColor(ColorHelper.html2color(Main.pref.get("color.scale", "#ffffff")));
     37                g.setColor(Main.pref.getColor(marktr("scale"), Color.white));
    3538                g.drawLine(0, 5, 99, 5);
    3639                g.drawLine(0, 0, 0, 10);

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

    r775 r804  
    215215
    216216                Graphics2D tempG = offscreenBuffer.createGraphics();
    217                 tempG.setColor(Preferences.getPreferencesColor("background", Color.BLACK));
     217                tempG.setColor(Main.pref.getColor("background", Color.BLACK));
    218218                tempG.fillRect(0, 0, getWidth(), getHeight());
    219219

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

    r758 r804  
    137137         */
    138138        public void paintConflicts(final Graphics g, final NavigatableComponent nc) {
    139                 Color preferencesColor = Preferences.getPreferencesColor("conflict", Color.gray);
     139                Color preferencesColor = Main.pref.getColor("conflict", Color.gray);
    140140                if (preferencesColor.equals(Color.BLACK))
    141141                        return;

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

    r803 r804  
    135135                        }
    136136                }
     137                if(s1.length() == 0 && s2.length() != 0)
     138                        return 1;
     139                else if(s2.length() == 0 && s1.length() != 0)
     140                        return -1;
    137141
    138142                // Default handling if the role name is nothing like "stop_xx"
    … …  
    152156                        int roleResult = compareRole(r1.role, r2.role);
    153157
    154                         if (roleResult == 0) {
    155                                 return compareMemebers(r1.member, r2.member);
    156                         }
     158                        if (roleResult == 0)
     159                                roleResult = compareMemebers(r1.member, r2.member);
    157160
    158161                        return roleResult;

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

    r768 r804  
    33package org.openstreetmap.josm.gui.layer;
    44
     5import static org.openstreetmap.josm.tools.I18n.marktr;
    56import static org.openstreetmap.josm.tools.I18n.tr;
    67import static org.openstreetmap.josm.tools.I18n.trn;
    … …  
    135136                color.addActionListener(new ActionListener() {
    136137                        public void actionPerformed(ActionEvent e) {
    137                                 String col = Main.pref.get("color.layer "+name, Main.pref.get("color.gps point", ColorHelper.color2html(Color.gray)));
    138                                 JColorChooser c = new JColorChooser(ColorHelper.html2color(col));
     138                                JColorChooser c = new JColorChooser(Main.pref.getColor(marktr("gps point"), "layer "+name, Color.gray));
    139139                                Object[] options = new Object[]{tr("OK"), tr("Cancel"), tr("Default")};
    140                                 int answer = JOptionPane.showOptionDialog(Main.parent, c, tr("Choose a color"), JOptionPane.OK_CANCEL_OPTION, JOptionPane.PLAIN_MESSAGE, null, options, options[0]);
     140                                int answer = JOptionPane.showOptionDialog(Main.parent, c, tr("Choose a color"), JOptionPane.OK_CANCEL_OPTION,
     141                                JOptionPane.PLAIN_MESSAGE, null, options, options[0]);
    141142                                switch (answer) {
    142143                                case 0:
    143                                         Main.pref.put("color.layer "+name, ColorHelper.color2html(c.getColor()));
     144                                        Main.pref.putColor("layer "+name, c.getColor());
    144145                                        break;
    145146                                case 1:
    146147                                        return;
    147148                                case 2:
    148                                         Main.pref.put("color.layer "+name, null);
     149                                        Main.pref.putColor("layer "+name, null);
    149150                                        break;
    150151                                }
    … …  
    367368                 ****************************************************************/
    368369                Long startTime = System.currentTimeMillis();
    369                 String gpsCol = Main.pref.get("color.gps point");
    370                 String gpsColSpecial = Main.pref.get("color.layer "+name);
    371                 Color neutralColor;
    372                 if (!gpsColSpecial.equals("")) {
    373                         neutralColor = ColorHelper.html2color(gpsColSpecial);
    374                 } else if (!gpsCol.equals("")) {
    375                         neutralColor = ColorHelper.html2color(gpsCol);
    376                 } else{
    377                         neutralColor = Color.GRAY;
    378                 }
     370                Color neutralColor = Main.pref.getColor(marktr("gps point"), "layer "+name, Color.GRAY);
    379371                boolean forceLines = Main.pref.getBoolean("draw.rawgps.lines.force");                     // also draw lines between points belonging to different segments
    380372                boolean direction = Main.pref.getBoolean("draw.rawgps.direction");                        // draw direction arrows on the lines
    … …  
    399391                                           (!neutralColor.equals(computeCacheColorUsed)) ||
    400392                                           (computeCacheColored != colored))) {
    401                         System.out.println("(re-)computing gpx line styles, reason: CCIS=" + computeCacheInSync + " CCMLLU=" + (computeCacheMaxLineLengthUsed != maxLineLength) + " CCCU=" +  (!neutralColor.equals(computeCacheColorUsed)) + " CCC=" + (computeCacheColored != colored));
     393//                      System.out.println("(re-)computing gpx line styles, reason: CCIS=" + computeCacheInSync + " CCMLLU=" + (computeCacheMaxLineLengthUsed != maxLineLength) + " CCCU=" +  (!neutralColor.equals(computeCacheColorUsed)) + " CCC=" + (computeCacheColored != colored));
    402394                        computeCacheMaxLineLengthUsed = maxLineLength;
    403395                        computeCacheInSync = false;
    … …  
    524516                 ********** STEP 3d - DRAW LARGE POINTS *************************
    525517                 ****************************************************************/
    526                                                 if (large) {
     518                if (large) {
    527519                        g.setColor(neutralColor);
    528520                        for (GpxTrack trk : data.tracks) {

  • trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

    r795 r804  
    157157                                        Point p1 = mv.getPoint(en1);
    158158                                        Point p2 = mv.getPoint(en2);
    159                                         Color color = inactive ? Preferences.getPreferencesColor(marktr("inactive"), Color.DARK_GRAY) :
    160                                                         Preferences.getPreferencesColor(marktr("downloaded Area"), Color.YELLOW);
     159                                        Color color = inactive ? Main.pref.getColor(marktr("inactive"), Color.DARK_GRAY) :
     160                                                        Main.pref.getColor(marktr("downloaded Area"), Color.YELLOW);
    161161                                        g.setColor(color);
    162162                                        g.drawRect(Math.min(p1.x,p2.x), Math.min(p1.y, p2.y), Math.abs(p2.x-p1.x), Math.abs(p2.y-p1.y));

  • trunk/src/org/openstreetmap/josm/gui/layer/RawGpsLayer.java

    r627 r804  
    22package org.openstreetmap.josm.gui.layer;
    33
     4import static org.openstreetmap.josm.tools.I18n.marktr;
    45import static org.openstreetmap.josm.tools.I18n.tr;
    56import static org.openstreetmap.josm.tools.I18n.trn;
    … …  
    227228
    228229        @Override public void paint(Graphics g, MapView mv) {
    229                 String gpsCol = Main.pref.get("color.gps point");
    230                 String gpsColSpecial = Main.pref.get("color.layer "+name);
    231                 if (!gpsColSpecial.equals(""))
    232                         g.setColor(ColorHelper.html2color(gpsColSpecial));
    233                 else if (!gpsCol.equals(""))
    234                         g.setColor(ColorHelper.html2color(gpsCol));
    235                 else
    236                         g.setColor(Color.GRAY);
     230                g.setColor(Main.pref.getColor(marktr("gps point"), "layer "+name, Color.gray));
    237231                Point old = null;
    238232
    … …  
    329323                color.addActionListener(new ActionListener(){
    330324                        public void actionPerformed(ActionEvent e) {
    331                                 String col = Main.pref.get("color.layer "+name, Main.pref.get("color.gps point", ColorHelper.color2html(Color.gray)));
    332                                 JColorChooser c = new JColorChooser(ColorHelper.html2color(col));
     325                                JColorChooser c = new JColorChooser(Main.pref.getColor(marktr("gps point"), "layer "+name, Color.gray));
    333326                                Object[] options = new Object[]{tr("OK"), tr("Cancel"), tr("Default")};
    334                                 int answer = JOptionPane.showOptionDialog(Main.parent, c, tr("Choose a color"), JOptionPane.OK_CANCEL_OPTION, JOptionPane.PLAIN_MESSAGE, null, options, options[0]);
     327                                int answer = JOptionPane.showOptionDialog(Main.parent, c, tr("Choose a color"), JOptionPane.OK_CANCEL_OPTION,
     328                                JOptionPane.PLAIN_MESSAGE, null, options, options[0]);
    335329                                switch (answer) {
    336330                                case 0:
    337                                         Main.pref.put("color.layer "+name, ColorHelper.color2html(c.getColor()));
     331                                        Main.pref.putColor("layer "+name, c.getColor());
    338332                                        break;
    339333                                case 1:
    340334                                        return;
    341335                                case 2:
    342                                         Main.pref.put("color.layer "+name, null);
     336                                        Main.pref.putColor("layer "+name, null);
    343337                                        break;
    344338                                }

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

    r794 r804  
    22package org.openstreetmap.josm.gui.layer.markerlayer;
    33
     4import static org.openstreetmap.josm.tools.I18n.marktr;
    45import static org.openstreetmap.josm.tools.I18n.tr;
    56import static org.openstreetmap.josm.tools.I18n.trn;
    … …  
    135136                boolean mousePressedTmp = mousePressed;
    136137                Point mousePos = mv.getMousePosition();
    137                 String mkrCol = Main.pref.get("color.gps marker");
    138                 String mkrColSpecial = Main.pref.get("color.layer "+name);
    139         String mkrTextShow = Main.pref.get("marker.show "+name, "show");
    140 
    141                 if (!mkrColSpecial.equals(""))
    142                         g.setColor(ColorHelper.html2color(mkrColSpecial));
    143                 else if (!mkrCol.equals(""))
    144                         g.setColor(ColorHelper.html2color(mkrCol));
    145                 else
    146                         g.setColor(Color.GRAY);
     138                String mkrTextShow = Main.pref.get("marker.show "+name, "show");
     139
     140                g.setColor(Main.pref.getColor(marktr("gps marker"), "layer "+name, Color.gray));
    147141               
    148142                for (Marker mkr : data) {
    … …  
    183177                color.addActionListener(new ActionListener(){
    184178                        public void actionPerformed(ActionEvent e) {
    185                                 String col = Main.pref.get("color.layer "+name, Main.pref.get("color.gps marker", ColorHelper.color2html(Color.gray)));
    186                                 JColorChooser c = new JColorChooser(ColorHelper.html2color(col));
     179                                JColorChooser c = new JColorChooser(Main.pref.getColor(marktr("gps marker"), "layer "+name, Color.gray));
    187180                                Object[] options = new Object[]{tr("OK"), tr("Cancel"), tr("Default")};
    188                                 int answer = JOptionPane.showOptionDialog(Main.parent, c, tr("Choose a color"), JOptionPane.OK_CANCEL_OPTION, JOptionPane.PLAIN_MESSAGE, null, options, options[0]);
     181                                int answer = JOptionPane.showOptionDialog(Main.parent, c, tr("Choose a color"), JOptionPane.OK_CANCEL_OPTION,
     182                                JOptionPane.PLAIN_MESSAGE, null, options, options[0]);
    189183                                switch (answer) {
    190184                                case 0:
    191                                         Main.pref.put("color.layer "+name, ColorHelper.color2html(c.getColor()));
     185                                        Main.pref.putColor("layer "+name, c.getColor());
    192186                                        break;
    193187                                case 1:
    194188                                        return;
    195189                                case 2:
    196                                         Main.pref.put("color.layer "+name, null);
     190                                        Main.pref.putColor("layer "+name, null);
    197191                                        break;
    198192                                }

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

    r798 r804  
    151151                        marktr("relation"), ColorHelper.color2html(SimplePaintVisitor.teal),
    152152                        marktr("selected"), ColorHelper.color2html(Color.white),
     153                        marktr("gps marker"), ColorHelper.color2html(Color.gray),
    153154                        marktr("gps point"), ColorHelper.color2html(Color.gray),
    154155                        marktr("conflict"), ColorHelper.color2html(Color.gray),
    … …  
    157158                };
    158159                for (int i = 0; i < cp.length/2; ++i)
     160                {
    159161                        if (!Main.pref.hasKey("color."+cp[i*2]))
    160162                                Main.pref.put("color."+cp[i*2], cp[i*2+1]);
     163                        Main.pref.putDefault("color."+cp[i*2], cp[i*2+1]);
     164                }
    161165    }
    162166

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

    r627 r804  
    6161
    6262                addTab(null, ImageProvider.get("preferences", icon), p);
    63                 setToolTipTextAt(getTabCount()-1, desc);
     63                setToolTipTextAt(getTabCount()-1, "<html>"+desc+"</html>");
    6464                return p;
    6565        }
Note: See TracChangeset for help on using the changeset viewer.