Hi,

I sent a request earlier asking if there's a way to prevent paragraphs from 
being defined as <div class="p"> and instead <p>, but it was rejected as not 
being a bug report.

Looking back, this may be a bug if you consider screen readers. A screen reader 
will not know that <div class="p"> is really a paragraph, and likewise the fake 
bulleted lists generated from related links - they should be real un-ordered 
lists for screen readers.

See below the changes I made.

Thanks,
Weldon

Modified:
   local/ditac/trunk/xsl/xhtml/body.xsl
   local/ditac/trunk/xsl/xhtml/relatedLinks.xsl

Modified: local/ditac/trunk/xsl/xhtml/body.xsl
==============================================================================
--- local/ditac/trunk/xsl/xhtml/body.xsl        Thu Jan 17 13:56:42 2013        
(r4822)
+++ local/ditac/trunk/xsl/xhtml/body.xsl        Thu Jan 17 14:05:26 2013        
(r4823)
@@ -19,11 +19,12 @@

   <xsl:template match="*[contains(@class,' topic/p ')]">
     <!-- a p can contain elements such as ul! -->
-    <div>
+    <!-- wsams (2013-01-16): The <p>'s in this template used to be <div>'s. -->
+    <p>
       <xsl:call-template name="commonAttributes"/>
       <xsl:call-template name="namedAnchor"/>
       <xsl:apply-templates/>
-    </div>
+    </p>
   </xsl:template>

   <!-- note ============================================================== -->

Modified: local/ditac/trunk/xsl/xhtml/relatedLinks.xsl
==============================================================================
--- local/ditac/trunk/xsl/xhtml/relatedLinks.xsl        Thu Jan 17 13:56:42 
2013        (r4822)
+++ local/ditac/trunk/xsl/xhtml/relatedLinks.xsl        Thu Jan 17 14:05:26 
2013        (r4823)
@@ -17,7 +17,8 @@
   <!-- link ============================================================== -->

   <xsl:template match="*[contains(@class,' topic/link ')]">
-    <p>
+    <!-- wsams (2013-01-16): The <li>'s in this template used to be <p>'s. -->
+    <li>
       <xsl:call-template name="commonAttributes"/>
       <xsl:call-template name="namedAnchor"/>
       <xsl:call-template name="linkItem"/>
@@ -47,11 +48,12 @@
               select="./*[contains(@class,' topic/linktext ')]"/>
         </xsl:otherwise>
       </xsl:choose>
-    </p>
+    </li>
   </xsl:template>

   <xsl:template name="linkItem">
-    <xsl:text>&#x2023; </xsl:text>
+    <!-- wsams: This is the fake bullet. -->
+    <!-- <xsl:text>&#x2023; </xsl:text> -->

     <xsl:if test="@role = 'parent' or
                   @role = 'previous' or
@@ -188,11 +190,12 @@
   <!-- linklist ========================================================== -->

   <xsl:template match="*[contains(@class,' topic/linklist ')]">
-    <div>
+    <!-- wsams (2013-01-16): The <ul>'s in this template used to be <div>'s. 
-->
+    <ul>
       <xsl:call-template name="commonAttributes"/>
       <xsl:call-template name="namedAnchor"/>
       <xsl:apply-templates/>
-    </div>
+    </ul>
   </xsl:template>

   <!-- linkinfo ========================================================== -->
--
XMLmind DITA Converter Support List
[email protected]
http://www.xmlmind.com/mailman/listinfo/ditac-support

Reply via email to