I made a small fix to match the JDK and I took out some inefficent code.
2006-01-03 Lillian Angel <[EMAIL PROTECTED]>
* javax/swing/JList.java
(init): visibleRowCount should be 7, like the JDK.
* javax/swing/plaf/metal/MetalFileChooserUI.java
(installComponents): No need to add the fileFilterCombo
to a panel. It can be added to the row directly.
Index: javax/swing/JList.java
===================================================================
RCS file: /cvsroot/classpath/classpath/javax/swing/JList.java,v
retrieving revision 1.42
diff -u -r1.42 JList.java
--- javax/swing/JList.java 28 Nov 2005 18:33:39 -0000 1.42
+++ javax/swing/JList.java 3 Jan 2006 16:11:11 -0000
@@ -1070,7 +1070,7 @@
layoutOrientation = VERTICAL;
opaque = true;
valueIsAdjusting = false;
- visibleRowCount = 8;
+ visibleRowCount = 7;
cellRenderer = new DefaultListCellRenderer();
listListener = new ListListener();
Index: javax/swing/plaf/metal/MetalFileChooserUI.java
===================================================================
RCS file: /cvsroot/classpath/classpath/javax/swing/plaf/metal/MetalFileChooserUI.java,v
retrieving revision 1.14
diff -u -r1.14 MetalFileChooserUI.java
--- javax/swing/plaf/metal/MetalFileChooserUI.java 3 Jan 2006 15:29:52 -0000 1.14
+++ javax/swing/plaf/metal/MetalFileChooserUI.java 3 Jan 2006 16:11:12 -0000
@@ -1330,11 +1330,9 @@
row1.add(fileNamePanel);
bottomPanel.add(row1);
- JPanel filterPanel = new JPanel(new VerticalMidLayout());
- filterPanel.add(fileFilterCombo);
JPanel row2 = new JPanel(new BorderLayout());
row2.add(new JLabel(this.filterLabel), BorderLayout.WEST);
- row2.add(filterPanel);
+ row2.add(fileFilterCombo);
bottomPanel.add(row2);
JPanel buttonPanel = new JPanel(new ButtonLayout());
buttonPanel.setBorder(BorderFactory.createEmptyBorder(4, 4, 4, 0));
_______________________________________________
Classpath-patches mailing list
Classpath-patches@gnu.org
http://lists.gnu.org/mailman/listinfo/classpath-patches
