Date: prev next · Thread: first prev next last
2014 Archives by date, by thread · List index


From: Andrew Sayers <andrew-libreoffice.org@pileofstuff.org>

---
 .../export/common/styles/style_collector.xsl       |  2 +-
 filter/source/xslt/odf2xhtml/export/xhtml/body.xsl | 45 +++++++++++-----------
 .../source/xslt/odf2xhtml/export/xhtml/table.xsl   |  4 +-
 3 files changed, 25 insertions(+), 26 deletions(-)

diff --git filter/source/xslt/odf2xhtml/export/common/styles/style_collector.xsl 
filter/source/xslt/odf2xhtml/export/common/styles/style_collector.xsl
index 9b02d94..69d34a4 100644
--- filter/source/xslt/odf2xhtml/export/common/styles/style_collector.xsl
+++ filter/source/xslt/odf2xhtml/export/common/styles/style_collector.xsl
@@ -661,7 +661,7 @@
     
             <!-- for-each changes the key environment from the previously globalData back to the 
document root  -->
         <xsl:for-each select="$documentRoot">
-                <!-- only styles, which are used in the content are written as CSS styles -->
+                <!-- ignore styles that aren't used in the content -->
             <xsl:choose>
                 <xsl:when test="key('elementUsingStyle', $style/@style:name)/@* or 
key('listLabelStyleInContent', $style/@style:name)/@*">
                     <xsl:call-template name="writeUsedStyles2">
diff --git filter/source/xslt/odf2xhtml/export/xhtml/body.xsl 
filter/source/xslt/odf2xhtml/export/xhtml/body.xsl
index 3ecde5e..6f3e980 100644
--- filter/source/xslt/odf2xhtml/export/xhtml/body.xsl
+++ filter/source/xslt/odf2xhtml/export/xhtml/body.xsl
@@ -433,8 +433,8 @@
                        <xsl:otherwise>
                                <xsl:choose>
                                        <!-- !!Check if paragraph is empty!!
-                                               OOo writes out empty paragraphs layouted behind an 
image (= draw:image within draw:frame)
-                                               those have to be neglected in HTML -->
+                                               OOo writes out empty paragraphs laid out behind an 
image (= draw:image within draw:frame),
+                                               those must be neglected in HTML -->
                                        <xsl:when test="name() = 'text:p' and not($childText) and 
not(*)">
                                                <xsl:variable name="isFollowingImage">
                                                        <xsl:call-template 
name="follows-empty-paragraphs-and-image">
@@ -479,7 +479,7 @@
 
 
                <!--
-                       Every tabulator indents its following content, encapuslated in a span
+                       Every tabulator indents its following content, contained in a span
                        element.
 
                        This template have two modes:
@@ -499,7 +499,7 @@
 
                <xsl:choose>
                        <xsl:when test="name() = 'text:tab'">
-                               <!-- every frame sibling have to be incapuslated within a div with 
left indent  -->
+                               <!-- every frame sibling must be contained within a left-indented 
div -->
                                <xsl:element name="span">
                                        <xsl:choose>
                                                <xsl:when test="count($tabStops/style:tab-stop) 
&gt; 0 and count($tabStops/style:tab-stop) &lt; 3">
@@ -630,15 +630,15 @@
                <xsl:apply-templates select="$tabStops/style:tab-stop[$tabCount]/@style:type"/>
        </xsl:template>
 
-       <!-- OOo writes out empty paragraphs layouted behind an image,
-               those have to be neglected in HTML
+       <!-- OOo writes out empty paragraphs laid out behind an image,
+               those must be neglected in HTML
        This method checks if an empty paragraph is of that kind! -->
        <xsl:template name="follows-empty-paragraphs-and-image">
                <xsl:param name="precedingElement" />
                <xsl:param name="elementToCheck" select="1"/>
                <xsl:choose>
-                       <!-- OOo writes out empty paragraphs layouted behind the image,
-                               those have to be neglected in HTML
+                       <!-- OOo writes out empty paragraphs laid out behind the image,
+                               those must be neglected in HTML
                        <xsl:when test="name() = 'text:p' and (normalize-space($childText) = '')">  
                       -->
                                <!-- WorkAround: Test if the empty paragraph was added after an 
image, which OOO often does -->
                        <xsl:when test="(name($precedingElement) = 'text:p' and 
not($precedingElement/text()) and not($precedingElement/*))">
@@ -729,8 +729,8 @@
                <!-- the footnote symbol is the prefix for a footnote in the footer -->
                <xsl:param name="footnotePrefix" />
 
-                       <!-- empty paragraph tags does not provoke an carridge return,
-                               therefore an non breakable space (&nbsp) have been inserted.-->
+                       <!-- empty paragraph tags do not cause a carriage return -
+                               insert a non-breakable space (&nbsp) to force the issue -->
                <xsl:choose>
                        <xsl:when test="node()">
                                <xsl:call-template name="apply-styles-and-content">
@@ -790,9 +790,9 @@
                                <xsl:variable name="nextSiblingIsFrame" 
select="name(following-sibling::node()[1]) = 'draw:frame'"/>
                                <xsl:choose>
                                        <xsl:when test="$createDiv and normalize-space(.) != ''">
-                                               <!-- every following frame sibling till the next 
draw:frame
-                                                       have to be incapuslated within a div with 
left indent.
-                                                       To be moved altogether arcording the indent 
(usually right) -->
+                                               <!-- every following frame sibling until the next 
<draw:frame>
+                                                       must be contained within a left-indented 
div, so they can
+                                                       be indented correctly (usually 
right-indented) -->
                                                <xsl:comment>Next 'div' added for 
floating.</xsl:comment>
                                                <xsl:element name="div">
                                                        <xsl:attribute name="style">
@@ -803,7 +803,7 @@
                                                        <xsl:apply-templates select=".">
                                                                <xsl:with-param name="globalData" 
select="$globalData"/>
                                                        </xsl:apply-templates>
-                                                       <!-- if it is a frame sibling it will be 
NOT incapuslated within the div (as already within one) -->
+                                                       <!-- if it is a frame sibling, it must not 
be contained within the div (as already within one) -->
                                                        <xsl:if test="not($nextSiblingIsFrame)">
                                                                <xsl:apply-templates 
select="following-sibling::node()[1]" mode="frameFloating">
                                                                        <xsl:with-param 
name="globalData" select="$globalData"/>
@@ -1325,7 +1325,7 @@
        and preceding siblings of 'text:h' with the same 'text:outline-level' (until a 
text:outline-level with lower value is found).
        If the 'text:start-value is not set the default value of '1' has to be taken.
        If a heading number is found (e.g. text:outline-level='3') all heading numbers
-       for the higher levels have to be written out -->
+       for the higher levels must be written out -->
        <xsl:template name="calc-heading-number">
                <xsl:param name="globalData"/>
                <xsl:param name="outlineLevel"/><!-- text level of the heading -->
@@ -2004,10 +2004,9 @@
                                                                                        
</xsl:otherwise>
                                                                                </xsl:choose>
                                                                        </xsl:variable>
-                                                                       <!-- Numbering is being 
done by this transformation creating a HTML span representing the number label
-                                                                                The html:span 
represents the list item/header label (e.g. 1.A.III)
-                                                                                As the html:span 
is usually a inline element is formated by CSS as block element to use width upon it,
-                                                                                to disable the 
caridge return float:left is used and later neglected -->
+                                                                       <!-- Create an HTML span to 
represent the label for a numbered list item
+                                                                                The <html:span> 
represents the list item/header label (e.g. 1.A.III).
+                                                                                We use CSS to make 
the <span> a block element, to set its width, and to float in the correct place -->
                                                                        <xsl:element name="span">
                                                                                <xsl:if 
test="$listLevelStyle/@text:style-name">
                                                                                        
<xsl:attribute name="class">
@@ -2046,8 +2045,8 @@
                                                                                                
</xsl:otherwise>
                                                                                        
</xsl:choose>
                                                                                </xsl:variable>
-                                                                               <!-- Some browsers 
have problem with stand-alone elements (e.g. <span/>)
-                                                                                       Therefore a 
comment is being inserted into an empty label -->
+                                                                               <!-- Some browsers 
have problems with stand-alone elements (i.e. <span/>),
+                                                                                       so we use a 
non-breaking space instead -->
                                                                                <xsl:choose>
                                                                                        <xsl:when 
test="$labelContent != ''">
                                                                                                
<xsl:value-of select="$labelContent"/>
@@ -2186,7 +2185,7 @@
 
 
        <!-- The Numbering start value (or offset from regular counteing) is used at the first item 
of offset,
-       but have to be reused on following items with no text:start-value -->
+       but must be reused on following items with no text:start-value -->
        <xsl:template name="getItemNumber">
                <xsl:param name="listLevel"/>
                <xsl:param name="listLevelStyle"/>
@@ -2204,7 +2203,7 @@
                </xsl:call-template>
        </xsl:template>
 
-       <!-- When there is a text:start-value the last have to be found and added to the number -->
+       <!-- When there is a text:start-value the last must be found and added to the number -->
        <xsl:template name="countListItemTillStartValue">
                <xsl:param name="IteratorSameLevelAndStyle" select="1"/>
                <xsl:param name="IteratorSameStyle" select="1"/>
diff --git filter/source/xslt/odf2xhtml/export/xhtml/table.xsl 
filter/source/xslt/odf2xhtml/export/xhtml/table.xsl
index 417e4f7..2f72ee7 100644
--- filter/source/xslt/odf2xhtml/export/xhtml/table.xsl
+++ filter/source/xslt/odf2xhtml/export/xhtml/table.xsl
@@ -158,8 +158,8 @@
                                <xsl:message>A table cell '<xsl:value-of select="$tableDataType" 
/>' element has been added!</xsl:message>
                        </xsl:if>
 
-                       <!-- empty cell tags produce problems with width CSS style on itself other 
table cells as well
-                               therefore an non breakable space (&nbsp;/&#160;) have been 
inserted.-->
+                       <!-- empty cell tags cause problems with with CSS styles (for the empty 
cells and others)
+                               therefore a non-breaking space (&nbsp;/&#160;) has been inserted -->
                        <xsl:choose>
                                <xsl:when test="node()">
                                        <xsl:call-template name="apply-styles-and-content">
-- 
1.8.5.1


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.