classpath-patches
[Top][All Lists]
Advanced

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

[cp-patches] FYI: fixed layout for JOptionPanes


From: Roman Kennke
Subject: [cp-patches] FYI: fixed layout for JOptionPanes
Date: Wed, 11 May 2005 14:00:28 +0200
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2.1) Gecko/20021204

I have added some layout stuff to JOptionPane based dialogs.

2005-05-11  Roman Kennke  <address@hidden>

       * javax/swing/plaf/basic/BasicOptionPaneUI.java
       (createSeparator): Implemented. Returns a vertical separator with
       a height of 17 pixels as recommended in the book Java Look & Feel
       Design Guidelines.
       (installComponents): Added an empty border around the dialog.


/Roman

Index: javax/swing/plaf/basic/BasicOptionPaneUI.java
===================================================================
RCS file: 
/cvsroot/classpath/classpath/javax/swing/plaf/basic/BasicOptionPaneUI.java,v
retrieving revision 1.14
diff -u -r1.14 BasicOptionPaneUI.java
--- javax/swing/plaf/basic/BasicOptionPaneUI.java       26 Apr 2005 18:56:19 
-0000      1.14
+++ javax/swing/plaf/basic/BasicOptionPaneUI.java       11 May 2005 11:57:10 
-0000
@@ -56,6 +56,7 @@
 import java.beans.PropertyChangeListener;
 import java.beans.PropertyVetoException;
 
+import javax.swing.BorderFactory;
 import javax.swing.Box;
 import javax.swing.BoxLayout;
 import javax.swing.Icon;
@@ -906,7 +907,7 @@
    */
   protected Container createSeparator()
   {
-    return null;
+    return (Container) Box.createVerticalStrut(17);
   }
 
   /**
@@ -1123,6 +1124,7 @@
        optionPane.add(button);
       }
 
+    optionPane.setBorder(BorderFactory.createEmptyBorder(12, 12, 11, 11));
     optionPane.invalidate();
   }
 

reply via email to

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