You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by an...@apache.org on 2007/02/02 15:31:28 UTC

svn commit: r502611 - /tapestry/tapestry4/trunk/tapestry-contrib/src/java/org/apache/tapestry/contrib/link/PopupLinkRenderer.java

Author: andyhot
Date: Fri Feb  2 06:31:27 2007
New Revision: 502611

URL: http://svn.apache.org/viewvc?view=rev&rev=502611
Log:
ajax friendly popuplinkrenderer

Modified:
    tapestry/tapestry4/trunk/tapestry-contrib/src/java/org/apache/tapestry/contrib/link/PopupLinkRenderer.java

Modified: tapestry/tapestry4/trunk/tapestry-contrib/src/java/org/apache/tapestry/contrib/link/PopupLinkRenderer.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry4/trunk/tapestry-contrib/src/java/org/apache/tapestry/contrib/link/PopupLinkRenderer.java?view=diff&rev=502611&r1=502610&r2=502611
==============================================================================
--- tapestry/tapestry4/trunk/tapestry-contrib/src/java/org/apache/tapestry/contrib/link/PopupLinkRenderer.java (original)
+++ tapestry/tapestry4/trunk/tapestry-contrib/src/java/org/apache/tapestry/contrib/link/PopupLinkRenderer.java Fri Feb  2 06:31:27 2007
@@ -74,15 +74,16 @@
 
         BodyBuilder builder = new BodyBuilder();
 
-        builder.addln("function {0}()", functionName);
+        builder.addln("{0}=function()", functionName);
         builder.begin();
-        builder.addln(
+        builder.add(
                 "var newWindow = window.open({0}, {1}, {2});",
                 TapestryUtils.enquote(url),
                 TapestryUtils.enquote(getWindowName()),
                 TapestryUtils.enquote(getFeatures()));
-        builder.addln("newWindow.focus();");
+        builder.add("newWindow.focus();");
         builder.end();
+        builder.addln(";");
 
         support.addBodyScript(component, builder.toString());