Changeset 9224 in josm for trunk/src/org


Ignore:
Timestamp:
2015-12-30T22:58:48+01:00 (4 years ago)
Author:
Don-vip
Message:

fix #12260 - adjust colors contrast in preferences tables (imagery, shortcuts, advanced preferences)

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

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/Preferences.java

    r9218 r9224  
    5050import javax.json.JsonWriter;
    5151import javax.swing.JOptionPane;
    52 import javax.swing.UIManager;
    5352import javax.xml.XMLConstants;
    5453import javax.xml.stream.XMLInputFactory;
     
    10121011    public synchronized Color getColor(String colName, Color def) {
    10131012        return getColor(colName, null, def);
    1014     }
    1015 
    1016     public synchronized Color getUIColor(String colName) {
    1017         return UIManager.getColor(colName);
    10181013    }
    10191014
  • trunk/src/org/openstreetmap/josm/gui/dialogs/LayerListDialog.java

    r9078 r9224  
    11151115                }
    11161116                if (c == null) {
    1117                     c = Main.pref.getUIColor(isSelected ? "Table.selectionForeground" : "Table.foreground");
     1117                    c = UIManager.getColor(isSelected ? "Table.selectionForeground" : "Table.foreground");
    11181118                }
    11191119                label.setForeground(c);
  • trunk/src/org/openstreetmap/josm/gui/preferences/advanced/PreferencesTable.java

    r9078 r9224  
    2424import javax.swing.JRadioButton;
    2525import javax.swing.JTable;
     26import javax.swing.UIManager;
    2627import javax.swing.table.DefaultTableCellRenderer;
    2728import javax.swing.table.DefaultTableModel;
     
    3435import org.openstreetmap.josm.data.Preferences.StringSetting;
    3536import org.openstreetmap.josm.gui.ExtendedDialog;
     37import org.openstreetmap.josm.gui.util.GuiHelper;
    3638import org.openstreetmap.josm.gui.widgets.JosmTextField;
    3739import org.openstreetmap.josm.tools.GBC;
     
    298300
    299301    private static class SettingCellRenderer extends DefaultTableCellRenderer {
    300         private final Color backgroundColor = Main.pref.getUIColor("Table.background");
     302        private final Color backgroundColor = UIManager.getColor("Table.background");
    301303        private final Color changedColor = Main.pref.getColor(
    302304                         marktr("Advanced Background: Changed"),
    303305                         new Color(200, 255, 200));
    304         private final Color foregroundColor = Main.pref.getUIColor("Table.foreground");
    305306        private final Color nonDefaultColor = Main.pref.getColor(
    306307                            marktr("Advanced Background: NonDefault"),
     
    319320                    display, isSelected, hasFocus, row, column);
    320321
    321             label.setBackground(backgroundColor);
    322             if (isSelected) {
    323                 label.setForeground(foregroundColor);
    324             }
     322            GuiHelper.setBackgroundReadable(label, backgroundColor);
    325323            if (pe.isChanged()) {
    326                 label.setBackground(changedColor);
     324                GuiHelper.setBackgroundReadable(label, changedColor);
    327325            } else if (!pe.isDefault()) {
    328                 label.setBackground(nonDefaultColor);
     326                GuiHelper.setBackgroundReadable(label, nonDefaultColor);
    329327            }
    330328
  • trunk/src/org/openstreetmap/josm/gui/preferences/imagery/ImageryPreference.java

    r9078 r9224  
    3737import javax.swing.JTable;
    3838import javax.swing.JToolBar;
     39import javax.swing.UIManager;
    3940import javax.swing.event.ListSelectionEvent;
    4041import javax.swing.event.ListSelectionListener;
     
    6263import org.openstreetmap.josm.gui.preferences.PreferenceSettingFactory;
    6364import org.openstreetmap.josm.gui.preferences.PreferenceTabbedPane;
     65import org.openstreetmap.josm.gui.util.GuiHelper;
    6466import org.openstreetmap.josm.gui.widgets.JosmEditorPane;
    6567import org.openstreetmap.josm.tools.GBC;
     
    248250                JLabel label = (JLabel) super.getTableCellRendererComponent(
    249251                        table, value, isSelected, hasFocus, row, column);
    250                 label.setBackground(Main.pref.getUIColor("Table.background"));
    251                 if (isSelected) {
    252                     label.setForeground(Main.pref.getUIColor("Table.foreground"));
    253                 }
     252                GuiHelper.setBackgroundReadable(label, UIManager.getColor("Table.background"));
    254253                if (value != null) { // Fix #8159
    255254                    String t = value.toString();
    256255                    for (ImageryInfo l : layers) {
    257256                        if (l.getExtendedUrl().equals(t)) {
    258                             label.setBackground(Main.pref.getColor(
     257                            GuiHelper.setBackgroundReadable(label, Main.pref.getColor(
    259258                                    marktr("Imagery Background: Default"),
    260259                                    new Color(200, 255, 200)));
     
    279278                JLabel label = (JLabel) super.getTableCellRendererComponent(
    280279                        table, info.getName(), isSelected, hasFocus, row, column);
    281                 label.setBackground(Main.pref.getUIColor("Table.background"));
    282                 if (isSelected) {
    283                     label.setForeground(Main.pref.getUIColor("Table.foreground"));
    284                 }
     280                GuiHelper.setBackgroundReadable(label, UIManager.getColor("Table.background"));
    285281                label.setToolTipText(info.getToolTipText());
    286282                return label;
  • trunk/src/org/openstreetmap/josm/gui/preferences/shortcut/PrefJPanel.java

    r9078 r9224  
    3333import javax.swing.RowFilter;
    3434import javax.swing.SwingConstants;
     35import javax.swing.UIManager;
    3536import javax.swing.event.DocumentEvent;
    3637import javax.swing.event.DocumentListener;
     
    4445
    4546import org.openstreetmap.josm.Main;
     47import org.openstreetmap.josm.gui.util.GuiHelper;
    4648import org.openstreetmap.josm.gui.widgets.JosmComboBox;
    4749import org.openstreetmap.josm.gui.widgets.JosmTextField;
     
    169171            JLabel label = (JLabel) super.getTableCellRendererComponent(
    170172                table, name ? sc.getLongText() : sc.getKeyText(), isSelected, hasFocus, row, column);
    171             label.setBackground(Main.pref.getUIColor("Table.background"));
    172             if (isSelected) {
    173                 label.setForeground(Main.pref.getUIColor("Table.foreground"));
    174             }
     173            GuiHelper.setBackgroundReadable(label, UIManager.getColor("Table.background"));
    175174            if (sc.isAssignedUser()) {
    176                 label.setBackground(Main.pref.getColor(
     175                GuiHelper.setBackgroundReadable(label, Main.pref.getColor(
    177176                        marktr("Shortcut Background: User"),
    178177                        new Color(200, 255, 200)));
    179178            } else if (!sc.isAssignedDefault()) {
    180                 label.setBackground(Main.pref.getColor(
     179                GuiHelper.setBackgroundReadable(label, Main.pref.getColor(
    181180                        marktr("Shortcut Background: Modified"),
    182181                        new Color(255, 255, 200)));
Note: See TracChangeset for help on using the changeset viewer.