Maven Doxia
  1. Maven Doxia
  2. DOXIA-48

Ordered lists always numbered 1,2,3

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0-alpha-6
    • Fix Version/s: 1.0-alpha-8
    • Component/s: None
    • Labels:
      None
    • Environment:
      osx 10.4.4 java 1.4.2_09
    • Number of attachments :
      1

      Description

      Ordered lists are supposed to be available as [[1]], [[a]], [[A]], [[i]], [[I]], but they all give numbers (ie the first option).

        Activity

        Hide
        Julian Wood added a comment -

        There are two ways to fix this bug. The css way, and the html way. I chose the html way.

        This patch simply observes the type of numbering demanded by the apt list ([[i]] for example), and writes the corresponding 'type' attribute on the ordered list.

        <ol type="i">

        The css way would be to write a class attribute instead, and then add corresponding styles in maven-base.css (or whichever stylesheet is the correct one). We could also do an inline style:

        <ol style="list-style: lower-roman">

        Both CSS solutions are much more flexible, but it looked like the XhtmlSink was designed to use the predefined html types, hence this quick solution.

        Show
        Julian Wood added a comment - There are two ways to fix this bug. The css way, and the html way. I chose the html way. This patch simply observes the type of numbering demanded by the apt list ([ [i] ] for example), and writes the corresponding 'type' attribute on the ordered list. <ol type="i"> The css way would be to write a class attribute instead, and then add corresponding styles in maven-base.css (or whichever stylesheet is the correct one). We could also do an inline style: <ol style="list-style: lower-roman"> Both CSS solutions are much more flexible, but it looked like the XhtmlSink was designed to use the predefined html types, hence this quick solution.

          People

          • Assignee:
            Brett Porter
            Reporter:
            Julian Wood
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: