Further testing some apps, I overlooked something really trivial. I have changed this patch completely. The problem was not the border, but the background color of the actual cell renderer (label) maybe set differently than that of the table. I fixed this.
2005-11-03 Lillian Angel <[EMAIL PROTECTED]> * javax/swing/table/DefaultTableCellRenderer.java (getTableCellRendererComponent): Fixed so that the label is painted with the correct background color. Does not depend on type of border. Removed this code. On Thu, 2005-11-03 at 14:38 -0500, Lillian Angel wrote: > If another class overrides getTableCellRendererComponent and sets the > border to be an empty border, the cell should be painted as if it > doesn't have focus at all times > > 2005-11-03 Lillian Angel <[EMAIL PROTECTED]> > > * javax/swing/table/DefaultTableCellRenderer.java > (getTableCellRendererComponent): Should only draw focus if the > border is not an empty border. > > _______________________________________________ > Classpath-patches mailing list > Classpath-patches@gnu.org > http://lists.gnu.org/mailman/listinfo/classpath-patches
Index: javax/swing/table/DefaultTableCellRenderer.java =================================================================== RCS file: /cvsroot/classpath/classpath/javax/swing/table/DefaultTableCellRenderer.java,v retrieving revision 1.17 diff -u -r1.17 DefaultTableCellRenderer.java --- javax/swing/table/DefaultTableCellRenderer.java 3 Nov 2005 19:39:05 -0000 1.17 +++ javax/swing/table/DefaultTableCellRenderer.java 3 Nov 2005 20:26:23 -0000 @@ -148,18 +148,14 @@ setBackground(table.getBackground()); setForeground(table.getForeground()); } - - Border b = UIManager.getBorder("Table.focusCellHighlightBorder"); - if (!(b instanceof EmptyBorder)) - { + if (hasFocus) { - setBackground(table.getBackground()); - setBorder(b); + setBackground(getBackground()); + setBorder(UIManager.getBorder("Table.focusCellHighlightBorder")); } else setBorder(BorderFactory.createEmptyBorder(1, 1, 1, 1)); - } setEnabled(table.isEnabled()); setFont(table.getFont());
_______________________________________________ Classpath-patches mailing list Classpath-patches@gnu.org http://lists.gnu.org/mailman/listinfo/classpath-patches