I committed this small fix to the MetalComboBoxEditor class. There is no need to recreate the editor component - it is created in the superclass constructor. Recreating it wipes out the custom setting for the number of columns in the JTextField. There is a new Mauve test to back this change:

2005-11-07  David Gilbert  <[EMAIL PROTECTED]>

        * javax/swing/plaf/metal/MetalComboBoxEditor.java
        (MetalComboBoxEditor): don't create new editor.

Regards,

Dave
Index: javax/swing/plaf/metal/MetalComboBoxEditor.java
===================================================================
RCS file: 
/cvsroot/classpath/classpath/javax/swing/plaf/metal/MetalComboBoxEditor.java,v
retrieving revision 1.3
diff -u -r1.3 MetalComboBoxEditor.java
--- javax/swing/plaf/metal/MetalComboBoxEditor.java     19 Oct 2005 13:46:02 
-0000      1.3
+++ javax/swing/plaf/metal/MetalComboBoxEditor.java     7 Nov 2005 12:03:52 
-0000
@@ -137,7 +137,6 @@
   public MetalComboBoxEditor()
   {
     super();
-    editor = new JTextField();
     editor.setBorder(new MetalComboBoxEditorBorder());
   }
   
_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to