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


Ignore:
Timestamp:
2012-06-24T21:15:57+02:00 (9 years ago)
Author:
Don-vip
Message:

conflict dialog: better display of empty rows in members tab, like previous work on nodes tab, plus fix Junit test

Location:
trunk/src/org/openstreetmap/josm/gui/conflict/pair
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListTableCellRenderer.java

    r5296 r5299  
    9393        setBackground(ConflictColors.BGCOLOR.get());
    9494        setForeground(ConflictColors.FGCOLOR.get());
     95        setBorder(null);
     96        setIcon(null);
    9597        setToolTipText(null);
    9698    }
  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/relation/RelationMemberTableCellRenderer.java

    r5266 r5299  
    153153        RelationMember member = (RelationMember)value;
    154154        reset();
    155         if (value == null)
    156             return this;
    157 
    158         renderBackground(getModel(table), member, row, column, isSelected);
    159         renderForeground(getModel(table), member, row, column, isSelected);
    160         switch(column) {
    161         case 0:
    162             renderRowId(row);
    163             break;
    164         case 1:
    165             if (member == null) {
    166                 renderEmptyRow();
    167             } else {
     155        if (member == null) {
     156            renderEmptyRow();
     157        } else {
     158            renderBackground(getModel(table), member, row, column, isSelected);
     159            renderForeground(getModel(table), member, row, column, isSelected);
     160            switch(column) {
     161            case 0:
     162                renderRowId(row);
     163                break;
     164            case 1:
    168165                renderRole(member);
     166                break;
     167            case 2:
     168                renderPrimitive(member);
     169                break;
     170            default:
     171                // should not happen
    169172            }
    170             break;
    171         case 2:
    172             if (member == null) {
    173                 renderEmptyRow();
    174             } else {
    175                 renderPrimitive(member);
    176             }
    177             break;
    178         default:
    179             // should not happen
    180173        }
    181174        return this;
Note: See TracChangeset for help on using the changeset viewer.