From: Matti Tahvonen <matti.tahvonen@itmill.com>
Date: Fri, 9 Jan 2009 17:24:33 +0000 (+0000)
Subject: improving windowing javadocs
X-Git-Tag: 6.7.0.beta1~3367
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=1355dd0fc925a79ba4cf53de76d85155e820c89e;p=vaadin-framework.git

improving windowing javadocs

svn changeset:6485/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/Application.java b/src/com/itmill/toolkit/Application.java
index a1ce33531c..fa4f5e9a9d 100644
--- a/src/com/itmill/toolkit/Application.java
+++ b/src/com/itmill/toolkit/Application.java
@@ -211,18 +211,27 @@ public abstract class Application implements URIHandler, Terminal.ErrorListener
      * method can be left as is. In case browser-level windows are needed, it is
      * recommended to create new window-objects on this method from their names
      * if the super.getWindow() does not find existing windows. See below for
-     * implementation example:
-     * <code>  // If we already have the requested window, use it
+     * implementation example: <code><pre>
+        // If we already have the requested window, use it
         Window w = super.getWindow(name);
         if (w == null) {
-
             // If no window found, create it
-            w = createNewWindow(name);
-        }
-        return w;</code>
+            w = new Window(name);
+            // set windows name to the one requested
+            w.setName(name);
+            // add it to this application
+            addWindow(w);
+            // add some content
+            w.addComponent(new Label("Test window"));
+        }
+        return w;</pre></code>
      * </p>
      * 
      * <p>
+     * <strong>Note</strong> that all returned Window objects must be added to
+     * this application instance.
+     * 
+     * <p>
      * The method should return null if the window does not exists (and is not
      * created as a side-effect) or if the application is not running anymore.
      * </p>