Mailing-List: contact forrest-dev-help@xml.apache.org; run by ezmlm
Precedence: bulk
Reply-To: forrest-dev@xml.apache.org
Subject: Re: Link to XML (Re: cvs commit: ...document2html.xsl)
From: Avik Sengupta <avik@apache.org>
To: forrest-dev@xml.apache.org
In-Reply-To: <20030318062711.GB4700@expresso.localdomain>
References: <20030317170357.47789.qmail@icarus.apache.org>
	<20030318062711.GB4700@expresso.localdomain>
Content-Type: text/plain
Content-Transfer-Encoding: 7bit
Date: 18 Mar 2003 12:04:15 +0530
Message-Id: <1047969283.4429.50.camel@IT-avik>
Mime-Version: 1.0

> So
> how about we have it switched off by default?  

Yes please... pretty please :)

On Tue, 2003-03-18 at 11:57, Jeff Turner wrote:
> On Mon, Mar 17, 2003 at 05:03:57PM -0000, nicolaken@apache.org wrote:
> > nicolaken    2003/03/17 09:03:56
> > 
> >   Modified:    src/resources/skins/common/xslt/html document2html.xsl
> >   Log:
> >   Enabling fixed version of the multiple output links on each page
> >   print - pdf - xml
> 
> Thing is, the XML link adds.. ~45% to the size of the generated output
> (3.1M -> 4.5M) and slows things down by 35% (2:11 -> 2:57).  From what I
> can see, being able to view the XML has nothing but curiousity value.  So
> how about we have it switched off by default?  
> 
> >   Still dunno how to disable them; I added attributes like for the PDF but
> >   dunno where it's set :-/
> 
> For a user-tweakable thing, an entry in skinconf.xml would be best:
> 
> <xsl:param name="config-file" select="'../../../../skinconf.xml'"/>
> <xsl:variable name="config" select="document($config-file)/skinconfig"/>
> ..
> <xsl:if test="$config/disable-xml-link = 'false'">
>   ...
> </xsl:if>
> 
> 
> --Jeff
>