Updated Panel, Window to use Component in constructors (#10300)

* Updated javadoc to be consistent with how Panel and Window behave (#10289)

Change-Id: I682d202e27b14d9695a8a6cd9eaff8a53fc6ae82

diff --git a/server/src/com/vaadin/ui/Panel.java b/server/src/com/vaadin/ui/Panel.java
index b5cfe044072432834b11112240ee71c624f868fd..472aefd5e0a12a4c07ee7cec660a8af10bfb39fd 100644 (file)
--- a/server/src/com/vaadin/ui/Panel.java
+++ b/server/src/com/vaadin/ui/Panel.java
@@ -56,20 +56,19 @@ public class Panel extends AbstractSingleComponentContainer implements
     };
 
     /**
-     * Creates a new empty panel. A VerticalLayout is used as content.
+     * Creates a new empty panel.
      */
     public Panel() {
         this((ComponentContainer) null);
     }
 
     /**
-     * Creates a new empty panel which contains the given content. The content
-     * cannot be null.
+     * Creates a new empty panel which contains the given content.
      * 
      * @param content
      *            the content for the panel.
      */
-    public Panel(ComponentContainer content) {
+    public Panel(Component content) {
         registerRpc(rpc);
         setContent(content);
         setWidth(100, Unit.PERCENTAGE);
@@ -77,7 +76,7 @@ public class Panel extends AbstractSingleComponentContainer implements
     }
 
     /**
-     * Creates a new empty panel with caption. Default layout is used.
+     * Creates a new empty panel with caption.
      * 
      * @param caption
      *            the caption used in the panel (HTML).
@@ -94,7 +93,7 @@ public class Panel extends AbstractSingleComponentContainer implements
      * @param content
      *            the content used in the panel.
      */
-    public Panel(String caption, ComponentContainer content) {
+    public Panel(String caption, Component content) {
         this(content);
         setCaption(caption);
     }

diff --git a/server/src/com/vaadin/ui/Window.java b/server/src/com/vaadin/ui/Window.java
index 3f6091aa9006f58b7d90734541b734171f3724bb..338c77239afea6ef72e1a8b9ef2d5ab8b1dd6fdd 100644 (file)
--- a/server/src/com/vaadin/ui/Window.java
+++ b/server/src/com/vaadin/ui/Window.java
@@ -42,10 +42,8 @@ import com.vaadin.shared.ui.window.WindowState;
  * {@link UI}. A window is added to a {@code UI} using
  * {@link UI#addWindow(Window)}. </p>
  * <p>
- * The contents of a window is set using {@link #setContent(ComponentContainer)}
- * or by using the {@link #Window(String, ComponentContainer)} constructor. The
- * contents can in turn contain other components. By default, a
- * {@link VerticalLayout} is used as content.
+ * The contents of a window is set using {@link #setContent(Component)} or by
+ * using the {@link #Window(String, Component)} constructor.
  * </p>
  * <p>
  * A window can be positioned on the screen using absolute coordinates (pixels)
@@ -75,14 +73,14 @@ public class Window extends Panel implements FocusNotifier, BlurNotifier,
     };
 
     /**
-     * Creates a new unnamed window with a default layout.
+     * Creates a new, empty sub window
      */
     public Window() {
         this("", null);
     }
 
     /**
-     * Creates a new unnamed window with a default layout and given title.
+     * Creates a new, empty sub window with a given title.
      * 
      * @param caption
      *            the title of the window.
@@ -92,14 +90,14 @@ public class Window extends Panel implements FocusNotifier, BlurNotifier,
     }
 
     /**
-     * Creates a new unnamed window with the given content and title.
+     * Creates a new, empty sub window with the given content and title.
      * 
      * @param caption
      *            the title of the window.
      * @param content
      *            the contents of the window
      */
-    public Window(String caption, ComponentContainer content) {
+    public Window(String caption, Component content) {
         super(caption, content);
         registerRpc(rpc);
         setSizeUndefined();