Maven Doxia
  1. Maven Doxia
  2. DOXIA-451

Tweak Doxia Markdown module HTML to better match

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.3
    • Fix Version/s: 1.3
    • Component/s: Module - Markdown
    • Labels:
      None
    • Flags:
      Patch

      Description

      The Doxia Markdown module currently uses the Pegdown module to generate HTML and then relies on the Doxia xhtml module to parse that. The Pegdown HTML generation currently produces HTML that doesn't exactly match what other modules produce, which causes some style errors. Specifically, for "code" blocks, there is no wrapping <div class="source"/> wrapper, which causes output to look strange. The attached patch adjusts the output.

      Of course, if the markdown module is going to be refactored to produce actual Doxia AST events, that might make this less of an issue. But I still think it'd be good to commit this patch in the meantime, especially if 1.3 is released before the refactoring.

        Activity

        Mark Thomas made changes -
        Assignee Lukas JOSEFIK [ lukas ] Lukas Theussl [ ltheussl ]
        Mark Thomas made changes -
        Workflow jira [ 12957002 ] Default workflow, editable Closed status [ 12993996 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 23:17:25 UTC 2015 [ 1428275845026 ]
        Mark Thomas made changes -
        Flags Patch [ 10430 ]
        Patch Submitted Yes [ 10763 ]
        Mark Thomas made changes -
        Workflow jira [ 12719861 ] Default workflow, editable Closed status [ 12748543 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 09:30:24 UTC 2015 [ 1428226224715 ]
        Lukas Theussl made changes -
        Field Original Value New Value
        Status Open [ 1 ] Closed [ 6 ]
        Fix Version/s 1.3 [ 17336 ]
        Resolution Fixed [ 1 ]
        Assignee Lukas Theussl [ lukas ]
        Brian Ferris created issue -

          People

          • Assignee:
            Lukas Theussl
            Reporter:
            Brian Ferris
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development