[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: Clean up copy-pasted code in fop1.xsl
Hi all,
The attached patch removes an unnecessary call for fop1.foxdest mode template which is a copy-paste from bookmark generation code. Should speed up XSLT a bit :)
As a related question, does it make sense to limit the template to "*[@id | @xml:id] instead of current match pattern? Given that IDs on subsequent runs are going to be different for elements without an explicit ID attribute, it won't be possible to create a permanent URL to point to such ID-less sections. On other hand, current match pattern does not generate destinations for tables/figures/equations, even if they have IDs - making impossible to <olink> to them.
Regards, Alexey. |
Index: fo/fop1.xsl =================================================================== --- fo/fop1.xsl (revision 663) +++ fo/fop1.xsl (working copy) @@ -107,16 +107,6 @@ <xsl:variable name="id"> <xsl:call-template name="object.id"/> </xsl:variable> - <xsl:variable name="bookmark-label"> - <xsl:apply-templates select="." mode="object.title.markup"/> - </xsl:variable> - <!--xsl:if test="$id != ''"> - <fox:destination internal-destination="{$id}"/> - </xsl:if--> - - <!-- Put the root element bookmark at the same level as its children --> - <!-- If the object is a set or book, generate a bookmark for the toc --> - <xsl:choose> <xsl:when test="self::index and $generate.index = 0"/> <xsl:when test="parent::*">
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]