classpath-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [cp-patches] FYI: JTable fixlet


From: Mark Wielaard
Subject: Re: [cp-patches] FYI: JTable fixlet
Date: Sat, 26 Nov 2005 14:12:49 +0100

Hi Roman,

On Thu, 2005-11-24 at 20:29 +0000, Roman Kennke wrote:
> I adjusted the DefaultTableCellRenderer to not adapt the JTables enabled
> property setting to the renderer. The cells are rendered normally even
> when the JTable itself is disabled.
> 
> 2005-11-24  Roman Kennke  <address@hidden>
> 
>         * javax/swing/table/DefaultTableCellRenderer.java
>         (getTableCellRendererComponent): Don't set enabled flag on the
>         renderer. The cells are rendered normally even when the table
>         is disabled.

This was also missing the attached (one-liner) patch.
Please do check that you actually attach the patches, that makes
reviewing a lot easier.

Thanks,

Mark
===================================================================
RCS file: 
/cvsroot/classpath/cvsroot/classpath/classpath/javax/swing/table/DefaultTableCellRenderer.java,v
retrieving revision 1.22
retrieving revision 1.23
diff -u -r1.22 -r1.23
--- classpath/classpath/javax/swing/table/DefaultTableCellRenderer.java 
2005/11/04 15:10:20     1.22
+++ classpath/classpath/javax/swing/table/DefaultTableCellRenderer.java 
2005/11/24 20:26:35     1.23
@@ -181,7 +181,6 @@
     else
       setBorder(BorderFactory.createEmptyBorder(1, 1, 1, 1));
 
-    setEnabled(table.isEnabled());
     setFont(table.getFont());
 
     // If the current background is equal to the table's background, then we

reply via email to

[Prev in Thread] Current Thread [Next in Thread]