Return-Path: 
 <dev-return-16702-apmail-forrest-dev-archive=forrest.apache.org@forrest.apache.org>
Delivered-To: apmail-forrest-dev-archive@www.apache.org
Received: (qmail 67853 invoked from network); 6 Jun 2005 00:40:53 -0000
Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199)
  by minotaur.apache.org with SMTP; 6 Jun 2005 00:40:52 -0000
Received: (qmail 52444 invoked by uid 500); 6 Jun 2005 00:40:52 -0000
Delivered-To: apmail-forrest-dev-archive@forrest.apache.org
Received: (qmail 52423 invoked by uid 500); 6 Jun 2005 00:40:51 -0000
Mailing-List: contact dev-help@forrest.apache.org; run by ezmlm
Precedence: bulk
list-help: <mailto:dev-help@forrest.apache.org>
list-unsubscribe: <mailto:dev-unsubscribe@forrest.apache.org>
List-Post: <mailto:dev@forrest.apache.org>
Reply-To: dev@forrest.apache.org
List-Id: <dev.forrest.apache.org>
Delivered-To: mailing list dev@forrest.apache.org
Received: (qmail 52407 invoked by uid 99); 6 Jun 2005 00:40:51 -0000
X-ASF-Spam-Status: No, hits=0.1 required=10.0
	tests=FORGED_RCVD_HELO
X-Spam-Check-By: apache.org
Received-SPF: neutral (hermes.apache.org: local policy)
Received: from natnoddy.rzone.de (HELO natnoddy.rzone.de) (81.169.145.166)
  by apache.org (qpsmtpd/0.28) with ESMTP; Sun, 05 Jun 2005 17:40:50 -0700
Received: from 192.168.2.2 (145.Red-83-39-17.pooles.rima-tde.net
 [83.39.17.145])
	by post.webmailer.de (8.13.1/8.13.1) with ESMTP id j560egns002597;
	Mon, 6 Jun 2005 02:40:43 +0200 (MEST)
Subject: Re: preview of new skin
From: Thorsten Scherler <thorsten@apache.org>
To: Diwaker Gupta <diwakergupta@gmail.com>
Cc: dev@forrest.apache.org
In-Reply-To: <1b0b45570506051619181b42b1@mail.gmail.com>
References: <1b0b4557050605121012cd10a0@mail.gmail.com>
	 <42A366AF.70607@apache.org> <1b0b45570506051504101a9bca@mail.gmail.com>
	 <42A385B3.6080909@apache.org>  <1b0b45570506051619181b42b1@mail.gmail.com>
Content-Type: text/plain
Date: Mon, 06 Jun 2005 02:40:41 +0200
Message-Id: <1118018441.5089.168.camel@localhost.localdomain>
Mime-Version: 1.0
X-Mailer: Evolution 2.0.2 
Content-Transfer-Encoding: 7bit
X-Virus-Checked: Checked
X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N

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)