Uploaded image for project: 'Maven Doxia'
  1. Maven Doxia
  2. DOXIA-451

Tweak Doxia Markdown module HTML to better match

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.3
    • 1.3
    • Module - Markdown
    • None
    • 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.

      Attachments

        Issue Links

          Activity

            People

              ltheussl Lukas Theussl
              bdferris@gmail.com Brian Ferris
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: