Is there some reason this doesn't also need the following?
markup.append("</html>");
-Dan
On Thu, Feb 28, 2008 at 7:08 AM, <[EMAIL PROTECTED]> wrote:
> Author: lindner
> Date: Thu Feb 28 07:07:49 2008
> New Revision: 631989
>
> URL: http://svn.apache.org/viewvc?rev=631989&view=rev
> Log:
> Ensure we generate a proper html doc, by correctly sending head and body
> tags
>
> Modified:
>
>
> incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/GadgetRenderingServlet.java
>
> Modified:
> incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/GadgetRenderingServlet.java
> URL:
> http://svn.apache.org/viewvc/incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/GadgetRenderingServlet.java?rev=631989&r1=631988&r2=631989&view=diff
>
> ==============================================================================
> ---
> incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/GadgetRenderingServlet.java
> (original)
> +++
> incubator/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/GadgetRenderingServlet.java
> Thu Feb 28 07:07:49 2008
> @@ -207,6 +207,7 @@
> markup.append("<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01//EN\"
> \"http://www.w3.org/TR/html4/strict.dtd\<http://www.w3.org/TR/html4/strict.dtd%5C>
> ">");
> }
>
> + markup.append("<html><head>");
> // TODO: This is so wrong.
> markup.append("<style type=\"text/css\">")
> .append("body,td,div,span,p{font-family:arial,sans-serif;}")
> @@ -214,6 +215,7 @@
> .append("a:active {color:#ff0000;}")
> .append("body{margin: 0px;padding:
> 0px;background-color:white;}")
> .append("</style>");
> + markup.append("</head><body>");
> StringBuilder externJs = new StringBuilder();
> StringBuilder inlineJs = new StringBuilder();
> String externFmt = "<script src=\"%s\"></script>";
>
>
>
