On Sun, 2005-06-05 at 16:19 -0700, Diwaker Gupta wrote:
> > > Actually something is terribly wrong with the recent versions of
> > > Firefox. They don't render the skin correctly. I have validated the
> > > XHTML on validator.w3.org and apart from the namespace issue, the skin
> > > validates *perfectly*. Konqueror has no problems rendering it either.
> > 
> > Hmmm... this prompted me to look at it in IE, and the problems are worse
> > there.
> 
> I have verified that (in both IE and Firefox) the problems are due to
> those vestigial namespace declarations in the html tag and the
> feedback tag. Everything works fine otherwise.
> 
> I had brought up this issue earlier, but never got around to fixing
> it. From that discussion it was apparent that these attributes are
> left-overs from the processing pipeline and certainly not required for
> the final output. Should I go ahead and look into getting rid of them?

I hope I fixed it in trunk (e.g. txt-link.ft):
Index:
org.apache.forrest.plugin.output.viewHelper.xhtml/resources/templates/txt-link.ft
===================================================================
---
org.apache.forrest.plugin.output.viewHelper.xhtml/resources/templates/txt-link.ft
       (revision 180112)
+++
org.apache.forrest.plugin.output.viewHelper.xhtml/resources/templates/txt-link.ft
       (working copy)
@@ -26,10 +26,8 @@
     <xsl:stylesheet version="1.1" 
       xmlns:xsl="http://www.w3.org/1999/XSL/Transform";>
       <xsl:template name="txt-link-body">
-        <div class="podlink" title="Plain Text Documentation">
-          <a href="{$filename-noext}.txt" class="dida"> <img
class="skin" 
-            src="{$skin-img-dir}/txtdoc.png" alt="TXT - icon" /><br/>
TXT</a>
-        </div>
+        <a href="{$filename-noext}.txt" class="format"> <img
class="skin" 
+          src="{$skin-img-dir}/txtdoc.png" alt="TXT - icon" /><div
class="caption">TXT</div></a>
       </xsl:template>
     </xsl:stylesheet>
   </forrest:template>

HTH

salu2
-- 
thorsten

"Together we stand, divided we fall!" 
Hey you (Pink Floyd)

Reply via email to