Return-Path: 
 <forrest-dev-return-2128-apmail-xml-forrest-dev-archive=xml.apache.org@xml.apache.org>
Delivered-To: apmail-xml-forrest-dev-archive@xml.apache.org
Received: (qmail 72964 invoked by uid 500); 30 Jul 2002 10:36:03 -0000
Mailing-List: contact forrest-dev-help@xml.apache.org; run by ezmlm
Precedence: bulk
list-help: <mailto:forrest-dev-help@xml.apache.org>
list-unsubscribe: <mailto:forrest-dev-unsubscribe@xml.apache.org>
list-post: <mailto:forrest-dev@xml.apache.org>
Reply-To: forrest-dev@xml.apache.org
Delivered-To: mailing list forrest-dev@xml.apache.org
Received: (qmail 72954 invoked from network); 30 Jul 2002 10:36:03 -0000
Received: from mail.hospitable.be (HELO relay.hospitable.be) (212.35.122.81)
  by daedalus.apache.org with SMTP; 30 Jul 2002 10:36:03 -0000
Received: from beke2000.vankets.com (5.104-136-217.adsl.skynet.be
 [217.136.104.5])
	by relay.hospitable.be (Postfix) with ESMTP id 5AC70A173C
	for <forrest-dev@xml.apache.org>; Tue, 30 Jul 2002 12:33:48 +0200 (CEST)
Message-Id: <5.1.0.14.0.20020730123613.03d64120@mail.visitronics.be>
X-Sender: postmaster.vankets@mail.visitronics.be
X-Mailer: QUALCOMM Windows Eudora Version 5.1
Date: Tue, 30 Jul 2002 12:38:08 +0200
To: forrest-dev@xml.apache.org
From: Bert Van Kets <bert@vankets.com>
Subject: RE: cvs commit: xml-forrest/src/resources/skins/bert/xslt/html
  si te2xhtml.xsl
In-Reply-To: <B6352CB87AA70442A1C6F49E6B658C9801EF3F@POSTMAN.flagship.ru
 >
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; format=flowed
X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N
Status: O
X-Status: 
X-Keywords: 
X-UID: 428

At 10:55 30/07/2002 +0400, you wrote:
> > From: Bert Van Kets [mailto:bert@vankets.com]
> >
> > I added a link to the favicon icon, so it would be copied.
> > At the same time I removed the 1pix image that contained the
> > link.  This
> > saves code and download time.
> > A <a href> is enough to get the images copied. It doesn not
> > have to enclose
> > an object.
> >
> > You might wonder why this list of links is necessary.
> > The images are used in CSS, but as CSS is not parsed those
> > images would not
> > be copied without the links.  Same is true for the favicon.
> > If there's no
> > link to the image/icon, it's not copied.
>
>IMHO, images should be copied by Ant and not by the Cocoon CLI. Of course,
>where/if it's possible.

I agree, but in the discussion we had earlier there was no easy way to do this.
If this can be solved it will solve some unnecessary skin directories in 
the sub directories.
BTW the favicon only works in the root of a site.  Are we going to keep it?

Bert


>Konstantin
>
> >
> > HTH,
> > Bert
> >
> >
> > At 12:33 30/07/2002 +1000, you wrote:
> > >I am not sure what was the purpose of this commit. It seems
> > >to have removed certain <img> tags, so leaving only links to
> > >images. Was that the intention? By the way, these changes have
> > >all been added to the forrest-site skin.
> > >--David
> > >
> > >bert@apache.org wrote:
> > > > bert        2002/07/29 12:59:59
> > > >
> > > >   Modified:    src/resources/skins/bert/xslt/html site2xhtml.xsl
> > > >   Log:
> > > >   Adding link to favicon.ico
> > > >
> > > >   Revision  Changes    Path
> > > >   1.12      +1
> > > -1
> > xml-forrest/src/resources/skins/bert/xslt/html/site2xhtml.xsl
> > > >
> > > >   Index: site2xhtml.xsl
> > > >
> > ===================================================================
> > > >   RCS file:
> > >
> > /home/cvs/xml-forrest/src/resources/skins/bert/xslt/html/site2
> > xhtml.xsl,v
> > > >   retrieving revision 1.11
> > > >   retrieving revision 1.12
> > > >   diff -u -r1.11 -r1.12
> > > >   --- site2xhtml.xsl  23 Jul 2002 08:26:18 -0000      1.11
> > > >   +++ site2xhtml.xsl  29 Jul 2002 19:59:59 -0000      1.12
> > > >   @@ -147,7 +147,7 @@
> > > >          <tr>
> > > >            <td width="10"><img
> > src="skin/images/spacer.gif" alt=""
> > > height="1" width="10" /></td>
> > > >          </tr>
> > > >   -      <tr><td bgcolor="#4C6C8F" height="1" colspan="2"><img
> > > src="skin/images/spacer.gif" alt="" width="1" height="1" /><a
> > > href="skin/images/label.gif"><img
> > src="skin/images/spacer.gif" alt=""
> > > width="1" height="1" border="0"/></a><a
> > href="skin/images/page.gif"><img
> > > src="skin/images/spacer.gif" alt="" width="1" height="1"
> > > border="0"/></a><a href="skin/images/chapter.gif"><img
> > > src="skin/images/spacer.gif" alt="" width="1" height="1"
> > > border="0"/></a><a href="skin/images/chapter_open.gif"><img
> > > src="skin/images/spacer.gif" alt="" width="1" height="1"
> > > border="0"/></a><a href="skin/images/current.gif"><img
> > > src="skin/images/spacer.gif" alt="" width="1" height="1"
> > > border="0"/></a></td></tr>
> > > >   +      <tr><td bgcolor="#4C6C8F" height="1" colspan="2"><img
> > > src="skin/images/spacer.gif" alt="" width="1" height="1" /><a
> > > href="skin/images/label.gif"></a><a
> > href="skin/images/page.gif"></a><a
> > > href="skin/images/chapter.gif"></a><a
> > > href="skin/images/chapter_open.gif"></a><a
> > > href="skin/images/current.gif"></a><a
> > href="/favicon.ico"></a></td></tr>
> > > >          <tr>
> > > >            <td align="center" class="copyright" bgcolor="#CFDCED"
> > > colspan="2">
> > > >              <font face="Arial, Helvetica, Sans-Serif"
> > > size="2">Copyright &#169; @year@ @vendor@. All Rights
> > Reserved<script
> > > language="JavaScript" type="text/javascript"><![CDATA[<!--
> > > >
> > > >
> > > >
> >