This patch (committed) fixes a minor problem I noticed in the MetalRadioButtonUI class. The look up keys for the UIDefaults table shouldn't be hard-coded, they should be constructed using the UI delegate's getPropertyPrefix():

2006-05-10  David Gilbert  <[EMAIL PROTECTED]>

        * javax/swing/plaf/metal/MetalRadioButtonUI.java
        (installDefaults): Use getPropertyPrefix() to allow subclasses to
        modify the lookup key.

Regards,

Dave
Index: javax/swing/plaf/metal/MetalRadioButtonUI.java
===================================================================
RCS file: 
/sources/classpath/classpath/javax/swing/plaf/metal/MetalRadioButtonUI.java,v
retrieving revision 1.9
diff -u -r1.9 MetalRadioButtonUI.java
--- javax/swing/plaf/metal/MetalRadioButtonUI.java      3 Jan 2006 15:29:52 
-0000       1.9
+++ javax/swing/plaf/metal/MetalRadioButtonUI.java      10 May 2006 20:45:10 
-0000
@@ -1,5 +1,5 @@
 /* MetalRadioButtonUI.java
-   Copyright (C) 2005 Free Software Foundation, Inc.
+   Copyright (C) 2005, 2006, Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -95,9 +95,10 @@
   public void installDefaults(AbstractButton b)
   {
     super.installDefaults(b);
-    disabledTextColor = UIManager.getColor("RadioButton.disabledText");
-    focusColor = UIManager.getColor("RadioButton.focus");
-    selectColor = UIManager.getColor("RadioButton.select");
+    String prefix = getPropertyPrefix();
+    disabledTextColor = UIManager.getColor(prefix + "disabledText");
+    focusColor = UIManager.getColor(prefix + "focus");
+    selectColor = UIManager.getColor(prefix + "select");
   }
   
   /**

Reply via email to