ni_nitdoc: Fixed quicksearch query with special characters
[nit.git] / src / ni_nitdoc.nit
index fb462cc..339a77f 100644 (file)
@@ -18,6 +18,7 @@ module ni_nitdoc
 
 import model_utils
 import modelize_property
+import markdown
 
 # The NitdocContext contains all the knowledge used for doc generation
 class NitdocContext
@@ -45,6 +46,9 @@ class NitdocContext
        private var opt_custom_overview_text: OptionString = new OptionString("Text displayed as introduction of Overview page before the modules list", "--custom-overview-text")
        private var opt_custom_footer_text: OptionString = new OptionString("Text displayed as footer of all pages", "--custom-footer-text")
 
+       private var opt_github_base: OptionString = new OptionString("The branch (or git ref) edited commits will be pulled into (ex: octocat:master)", "--github-base")
+       private var opt_github_head: OptionString = new OptionString("The reference branch name used to create pull requests (ex: master)", "--github-head")
+
        init do
                toolcontext.option_context.add_option(opt_dir)
                toolcontext.option_context.add_option(opt_source)
@@ -55,6 +59,8 @@ class NitdocContext
                toolcontext.option_context.add_option(opt_custom_footer_text)
                toolcontext.option_context.add_option(opt_custom_overview_text)
                toolcontext.option_context.add_option(opt_custom_menu_items)
+               toolcontext.option_context.add_option(opt_github_base)
+               toolcontext.option_context.add_option(opt_github_head)
                toolcontext.process_options
                self.arguments = toolcontext.option_context.rest
 
@@ -93,9 +99,9 @@ class NitdocContext
                else
                        var dir = "NIT_DIR".environ
                        if dir.is_empty then
-                               dir = "{sys.program_name.dirname}/../share/nitdoc"
+                               dir = "{sys.program_name.dirname}/../share/ni_nitdoc"
                        else
-                               dir = "{dir}/share/nitdoc"
+                               dir = "{dir}/share/ni_nitdoc"
                        end
                        share_dir = dir
                        if share_dir is null then
@@ -160,13 +166,13 @@ class NitdocContext
                file.write("var entries = \{ ")
                for mmodule in model.mmodules do
                        file.write("\"{mmodule.name}\": [")
-                       file.write("\{txt: \"{mmodule.name}\", url:\"{mmodule.url}\" \},")
+                       file.write("\{txt: \"{mmodule.full_name}\", url:\"{mmodule.url}\" \},")
                        file.write("],")
                end
                for mclass in model.mclasses do
                        if mclass.visibility < min_visibility then continue
                        file.write("\"{mclass.name}\": [")
-                       file.write("\{txt: \"{mclass.name}\", url:\"{mclass.url}\" \},")
+                       file.write("\{txt: \"{mclass.full_name}\", url:\"{mclass.url}\" \},")
                        file.write("],")
                end
                var name2mprops = new HashMap[String, Set[MPropDef]]
@@ -204,11 +210,13 @@ abstract class NitdocPage
                append("<meta charset='utf-8'/>")
                append("<script type='text/javascript' src='scripts/jquery-1.7.1.min.js'></script>")
                append("<script type='text/javascript' src='scripts/ZeroClipboard.min.js'></script>")
+               append("<script type='text/javascript' src='scripts/Markdown.Converter.js'></script>")
                append("<script type='text/javascript' src='quicksearch-list.js'></script>")
                append("<script type='text/javascript' src='scripts/base64.js'></script>")
                append("<script type='text/javascript' src='scripts/github.js'></script>")
                append("<script type='text/javascript' src='scripts/js-facilities.js'></script>")
                append("<link rel='stylesheet' href='styles/main.css' type='text/css' media='screen'/>")
+               append("<link rel='stylesheet' href='styles/github.css' type='text/css' media='screen'/>")
                var title = ""
                if ctx.opt_custom_title.value != null then
                        title = " | {ctx.opt_custom_title.value.to_s}"
@@ -281,7 +289,12 @@ abstract class NitdocPage
                append("<head>")
                head
                append("</head>")
-               append("<body>")
+               append("<body")
+               if not ctx.opt_github_base.value == null and not ctx.opt_github_head.value == null then
+                       append(" data-github-base='{ctx.opt_github_base.value.as(not null)}'")
+                       append(" data-github-head='{ctx.opt_github_head.value.as(not null)}'")
+               end
+               append(">")
                header
                append("<div class='page'>")
                content
@@ -523,18 +536,18 @@ class NitdocModule
        end
 
        redef fun title do
-               if mbuilder.mmodule2nmodule.has_key(mmodule) then
+               if mbuilder.mmodule2nmodule.has_key(mmodule) and not mbuilder.mmodule2nmodule[mmodule].short_comment.is_empty then
                        var nmodule = mbuilder.mmodule2nmodule[mmodule]
-                       return "{mmodule.name} module | {nmodule.short_comment}"
+                       return "{mmodule.html_name} module | {nmodule.short_comment}"
                else
-                       return "{mmodule.name} module"
+                       return "{mmodule.html_name} module"
                end
        end
 
        redef fun menu do
                super
                append("<li><a href='index.html'>Overview</a></li>")
-               append("<li class='current'>{mmodule.name}</li>")
+               append("<li class='current'>{mmodule.html_name}</li>")
                append("<li><a href='search.html'>Search</a></li>")
        end
 
@@ -609,16 +622,12 @@ class NitdocModule
 
        private fun module_doc do
                # title
-               append("<h1>{mmodule.name}</h1>")
+               append("<h1>{mmodule.html_name}</h1>")
                append("<div class='subtitle info'>")
                mmodule.html_signature(self)
                append("</div>")
                # comment
-               var nmodule = ctx.mbuilder.mmodule2nmodule[mmodule]
-               append("<section class='description'>")
-               if not nmodule.full_comment.is_empty then append("<pre>{nmodule.full_comment}</pre>")
-               process_generate_dot
-               append("</section>")
+               mmodule.html_comment(self)
                # classes
                var class_sorter = new MClassNameSorter
                # intro
@@ -766,9 +775,9 @@ class NitdocClass
        redef fun title do
                var nclass = ctx.mbuilder.mclassdef2nclassdef[mclass.intro]
                if nclass isa AStdClassdef then
-                       return "{mclass.name} class | {nclass.short_comment}"
+                       return "{mclass.html_name} class | {nclass.short_comment}"
                else
-                       return "{mclass.name} class"
+                       return "{mclass.html_name} class"
                end
        end
 
@@ -785,7 +794,7 @@ class NitdocClass
                        public_owner.html_link(self)
                        append("</li>")
                end
-               append("<li class='current'>{mclass.name}</li>")
+               append("<li class='current'>{mclass.html_name}</li>")
                append("<li><a href='search.html'>Search</a></li>")
        end
 
@@ -898,18 +907,15 @@ class NitdocClass
 
        private fun class_doc do
                # title
-               append("<h1>{mclass.name}{mclass.html_short_signature}</h1>")
+               append("<h1>{mclass.html_name}{mclass.html_short_signature}</h1>")
                append("<div class='subtitle info'>")
                if mclass.visibility < public_visibility then append("{mclass.visibility.to_s} ")
                append("{mclass.kind.to_s} ")
                mclass.html_namespace(self)
                append("{mclass.html_short_signature}</div>")
                # comment
-               var nclass = ctx.mbuilder.mclassdef2nclassdef[mclass.intro]
-               append("<section class='description'>")
-               if nclass isa AStdClassdef and not nclass.full_comment.is_empty then append("<pre>{nclass.full_comment}</pre>")
+               mclass.html_comment(self)
                process_generate_dot
-               append("</section>")
                # concerns
                var concern2meths = new ArrayMap[MModule, Array[MMethodDef]]
                var sorted_meths = new Array[MMethodDef]
@@ -939,18 +945,18 @@ class NitdocClass
                        var nowner = ctx.mbuilder.mmodule2nmodule[owner]
                        append("<li>")
                        if nowner.short_comment.is_empty then
-                               append("<a href=\"#{owner.anchor}\">{owner.name}</a>")
+                               append("<a href=\"#{owner.anchor}\">{owner.html_name}</a>")
                        else
-                               append("<a href=\"#{owner.anchor}\">{owner.name}</a>: {nowner.short_comment}")
+                               append("<a href=\"#{owner.anchor}\">{owner.html_name}</a>: {nowner.short_comment}")
                        end
                        if not mmodules.is_empty then
                                append("<ul>")
                                for mmodule in mmodules do
                                        var nmodule = ctx.mbuilder.mmodule2nmodule[mmodule]
                                        if nmodule.short_comment.is_empty then
-                                               append("<li><a href=\"#{mmodule.anchor}\">{mmodule.name}</a></li>")
+                                               append("<li><a href=\"#{mmodule.anchor}\">{mmodule.html_name}</a></li>")
                                        else
-                                               append("<li><a href=\"#{mmodule.anchor}\">{mmodule.name}</a>: {nmodule.short_comment}</li>")
+                                               append("<li><a href=\"#{mmodule.anchor}\">{mmodule.html_name}</a>: {nmodule.short_comment}</li>")
                                        end
                                end
                                append("</ul>")
@@ -962,6 +968,7 @@ class NitdocClass
                # properties
                var prop_sorter = new MPropDefNameSorter
                var lmmodule = new List[MModule]
+               var nclass = ctx.mbuilder.mclassdef2nclassdef[mclass.intro]
                # virtual and formal types
                var local_vtypes = new Array[MVirtualTypeDef]
                for vt in vtypes do if not inherited.has(vt) then local_vtypes.add(vt)
@@ -1116,8 +1123,12 @@ end
 #
 
 redef class MModule
+       # Return the HTML escaped name of the module
+       private fun html_name: String do return name.html_escape
+
        # URL to nitdoc page
-       fun url: String do
+       #       module_owner_name.html
+       private fun url: String do
                if url_cache == null then
                        var res = new Buffer
                        res.append("module_")
@@ -1132,8 +1143,9 @@ redef class MModule
        end
        private var url_cache: nullable String
 
-       # html anchor id to the module in a nitdoc page
-       fun anchor: String do
+       # html anchor id for the module in a nitdoc page
+       #       MOD_owner_name
+       private fun anchor: String do
                if anchor_cache == null then
                        var res = new Buffer
                        res.append("MOD_")
@@ -1149,13 +1161,14 @@ redef class MModule
        private var anchor_cache: nullable String
 
        # Return a link (html a tag) to the nitdoc module page
-       fun html_link(page: NitdocPage) do
+       #       <a href="url" title="short_comment">html_name</a>
+       private fun html_link(page: NitdocPage) do
                if html_link_cache == null then
                        var res = new Buffer
                        if page.ctx.mbuilder.mmodule2nmodule.has_key(self) then
-                               res.append("<a href='{url}' title='{page.ctx.mbuilder.mmodule2nmodule[self].short_comment}'>{name}</a>")
+                               res.append("<a href='{url}' title='{page.ctx.mbuilder.mmodule2nmodule[self].short_comment}'>{html_name}</a>")
                        else
-                               res.append("<a href='{url}'>{name}</a>")
+                               res.append("<a href='{url}'>{html_name}</a>")
                        end
                        html_link_cache = res.to_s
                end
@@ -1164,14 +1177,16 @@ redef class MModule
        private var html_link_cache: nullable String
 
        # Return the module signature decorated with html
-       fun html_signature(page: NitdocPage) do
+       #       <span>module html_full_namespace</span>
+       private fun html_signature(page: NitdocPage) do
                page.append("<span>module ")
                html_full_namespace(page)
                page.append("</span>")
        end
 
        # Return the module full namespace decorated with html
-       fun html_full_namespace(page: NitdocPage) do
+       #       <span>public_owner.html_namespace::html_link</span>
+       private fun html_full_namespace(page: NitdocPage) do
                page.append("<span>")
                var mowner = public_owner
                if mowner != null then
@@ -1183,7 +1198,8 @@ redef class MModule
        end
 
        # Return the module full namespace decorated with html
-       fun html_namespace(page: NitdocPage) do
+       #       <span>public_owner.html_namespace</span>
+       private fun html_namespace(page: NitdocPage) do
                page.append("<span>")
                var mowner = public_owner
                if mowner != null then
@@ -1195,12 +1211,23 @@ redef class MModule
        end
 
        # Return the full comment of the module decorated with html
-       fun html_full_comment(page: NitdocPage) do
+       private fun html_comment(page: NitdocPage) do
+               page.append("<div class='description'>")
                if page.ctx.mbuilder.mmodule2nmodule.has_key(self) then
-                       page.append("<div id='description'>")
-                       page.append("<pre>{page.ctx.mbuilder.mmodule2nmodule[self].full_comment}</pre>")
-                       page.append("</div>")
+                       var nmodule = page.ctx.mbuilder.mmodule2nmodule[self]
+                       page.append("<textarea class='baseComment' data-comment-namespace='{full_name}' data-comment-location='{nmodule.doc_location.to_s}'>{nmodule.full_comment}</textarea>")
+                       if nmodule.full_comment == "" then
+                               page.append("<p class='info inheritance'>")
+                               page.append("<span class=\"noComment\">no comment for </span>")
+                       else
+                               page.append("<div class='comment'>{nmodule.full_markdown}</div>")
+                               page.append("<p class='info inheritance'>")
+                       end
+                       page.append("definition in ")
+                       self.html_full_namespace(page)
+                       page.append(" {page.show_source(nmodule.location)}</p>")
                end
+               page.append("</div>")
        end
 
        private fun has_mclassdef_for(mclass: MClass): Bool do
@@ -1223,41 +1250,29 @@ redef class MModule
 end
 
 redef class MClass
-       # return the generic signature of the class
-       #       [E, F]
-       private fun html_short_signature: String do
-               if arity > 0 then
-                       return "[{intro.parameter_names.join(", ")}]"
-               else
-                       return ""
-               end
-       end
+       # Return the HTML escaped name of the module
+       private fun html_name: String do return name.html_escape
 
-       # return the generic signature of the class with bounds
-       #       [E: <a>MType</a>, F: <a>MType</a>]
-       private fun html_signature(page: NitdocPage) do
-               if arity > 0 then
-                       page.append("[")
-                       for i in [0..intro.parameter_names.length[ do
-                               page.append("{intro.parameter_names[i]}: ")
-                               intro.bound_mtype.arguments[i].html_link(page)
-                               if i < intro.parameter_names.length - 1 then page.append(", ")
-                       end
-                       page.append("]")
-               end
+       # URL to nitdoc page
+       #       class_owner_name.html
+       private fun url: String do
+               return "class_{public_owner}_{name}.html"
        end
 
-       # Return the class namespace decorated with html
-       private fun html_namespace(page: NitdocPage) do
-               intro_mmodule.html_namespace(page)
-               page.append("::<span>")
-               html_short_link(page)
-               page.append("</span>")
+       # html anchor id for the class in a nitdoc page
+       #       MOD_owner_name
+       private fun anchor: String do
+               if anchor_cache == null then
+                       anchor_cache = "CLASS_{public_owner.name}_{name}"
+               end
+               return anchor_cache.as(not null)
        end
+       private var anchor_cache: nullable String
 
-       # Return a link (html a tag) to the nitdoc class page
-       fun html_short_link(page: NitdocPage) do
-               if html_short_link_cache == null then
+       # Return a link (with signature) to the nitdoc class page
+       #       <a href="url" title="short_comment">html_name(signature)</a>
+       private fun html_link(page: NitdocPage) do
+               if html_link_cache == null then
                        var res = new Buffer
                        res.append("<a href='{url}'")
                        if page.ctx.mbuilder.mclassdef2nclassdef.has_key(intro) then
@@ -1266,16 +1281,17 @@ redef class MClass
                                        res.append(" title=\"{nclass.short_comment}\"")
                                end
                        end
-                       res.append(">{name}</a>")
-                       html_short_link_cache = res.to_s
+                       res.append(">{html_name}{html_short_signature}</a>")
+                       html_link_cache = res.to_s
                end
-               page.append(html_short_link_cache.as(not null))
+               page.append(html_link_cache.as(not null))
        end
-       private var html_short_link_cache: nullable String
+       private var html_link_cache: nullable String
 
-       # Return a link (html a tag) to the nitdoc class page
-       fun html_link(page: NitdocPage) do
-               if html_link_cache == null then
+       # Return a short link (without signature) to the nitdoc class page
+       #       <a href="url" title="short_comment">html_name</a>
+       private fun html_short_link(page: NitdocPage) do
+               if html_short_link_cache == null then
                        var res = new Buffer
                        res.append("<a href='{url}'")
                        if page.ctx.mbuilder.mclassdef2nclassdef.has_key(intro) then
@@ -1284,15 +1300,17 @@ redef class MClass
                                        res.append(" title=\"{nclass.short_comment}\"")
                                end
                        end
-                       res.append(">{name}{html_short_signature}</a>")
-                       html_link_cache = res.to_s
+                       res.append(">{html_name}</a>")
+                       html_short_link_cache = res.to_s
                end
-               page.append(html_link_cache.as(not null))
+               page.append(html_short_link_cache.as(not null))
        end
-       private var html_link_cache: nullable String
+       private var html_short_link_cache: nullable String
 
-       fun html_anchor(page: NitdocPage) do
-               if html_anchor_cache == null then
+       # Return a link (with signature) to the class anchor
+       #       <a href="url" title="short_comment">html_name</a>
+       private fun html_link_anchor(page: NitdocPage) do
+               if html_link_anchor_cache == null then
                        var res = new Buffer
                        res.append("<a href='#{anchor}'")
                        if page.ctx.mbuilder.mclassdef2nclassdef.has_key(intro) then
@@ -1301,38 +1319,57 @@ redef class MClass
                                        res.append(" title=\"{nclass.short_comment}\"")
                                end
                        end
-                       res.append(">{name}{html_short_signature}</a>")
-                       html_anchor_cache = res.to_s
+                       res.append(">{html_name}{html_short_signature}</a>")
+                       html_link_anchor_cache = res.to_s
                end
-               page.append(html_anchor_cache.as(not null))
+               page.append(html_link_anchor_cache.as(not null))
        end
-       private var html_anchor_cache: nullable String
+       private var html_link_anchor_cache: nullable String
 
-       fun anchor: String do
-               if anchor_cache == null then
-                       anchor_cache = "CLASS_{public_owner.name}_{name}"
+       # Return the generic signature of the class with bounds
+       #       [E: <a>MType</a>, F: <a>MType</a>]
+       private fun html_signature(page: NitdocPage) do
+               if arity > 0 then
+                       page.append("[")
+                       for i in [0..intro.parameter_names.length[ do
+                               page.append("{intro.parameter_names[i]}: ")
+                               intro.bound_mtype.arguments[i].html_link(page)
+                               if i < intro.parameter_names.length - 1 then page.append(", ")
+                       end
+                       page.append("]")
                end
-               return anchor_cache.as(not null)
        end
-       private var anchor_cache: nullable String
 
-       fun url: String do
-               return "class_{public_owner}_{name}.html"
+       # Return the generic signature of the class without bounds
+       #       [E, F]
+       private fun html_short_signature: String do
+               if arity > 0 then
+                       return "[{intro.parameter_names.join(", ")}]"
+               else
+                       return ""
+               end
        end
 
-       # Escape name for html output
-       redef fun name do return super.html_escape
+       # Return the class namespace decorated with html
+       #       <span>intro_module::html_short_link</span>
+       private fun html_namespace(page: NitdocPage) do
+               intro_mmodule.html_namespace(page)
+               page.append("::<span>")
+               html_short_link(page)
+               page.append("</span>")
+       end
 
        # Return a list item for the mclass
+       #       <li>html_link</li>
        private fun html_sidebar_item(page: NitdocModule) do
                if page.mmodule.in_nesting.greaters.has(intro.mmodule) then
                        page.append("<li class='intro'>")
                        page.append("<span title='Introduced'>I</span>")
-                       html_anchor(page)
+                       html_link_anchor(page)
                else if page.mmodule.has_mclassdef_for(self) then
                        page.append("<li class='redef'>")
                        page.append("<span title='Redefined'>R</span>")
-                       html_anchor(page)
+                       html_link_anchor(page)
                else
                        page.append("<li class='inherit'>")
                        page.append("<span title='Inherited'>H</span>")
@@ -1347,7 +1384,7 @@ redef class MClass
                if not page.mmodule.in_nesting.greaters.has(intro.mmodule) then classes.add("redef")
                classes.add(visibility.to_s)
                page.append("<article class='{classes.join(" ")}' id='{anchor}'>")
-               page.append("<h3 class='signature' data-untyped-signature='{name}{html_short_signature}'>")
+               page.append("<h3 class='signature' data-untyped-signature='{html_name}{html_short_signature}'>")
                page.append("<span>")
                html_short_link(page)
                html_signature(page)
@@ -1367,35 +1404,56 @@ redef class MClass
                page.append("{html_short_signature}</div>")
        end
 
-       private fun html_comment(page: NitdocModule) do
-               page.mmodule.linearize_mclassdefs(mclassdefs)
+       private fun html_comment(page: NitdocPage) do
                page.append("<div class='description'>")
-               # comments for each mclassdef contained in current mmodule
-               for mclassdef in mclassdefs do
-                       if not mclassdef.is_intro and not page.mmodule.mclassdefs.has(mclassdef) then continue
-                       if page.ctx.mbuilder.mclassdef2nclassdef.has_key(mclassdef) then
-                               var nclass = page.ctx.mbuilder.mclassdef2nclassdef[mclassdef]
+               if page isa NitdocModule then
+                       page.mmodule.linearize_mclassdefs(mclassdefs)
+                       # comments for each mclassdef contained in current mmodule
+                       for mclassdef in mclassdefs do
+                               if not mclassdef.is_intro and not page.mmodule.mclassdefs.has(mclassdef) then continue
+                               if page.ctx.mbuilder.mclassdef2nclassdef.has_key(mclassdef) then
+                                       var nclass = page.ctx.mbuilder.mclassdef2nclassdef[mclassdef]
+                                       if nclass isa AStdClassdef then
+                                               page.append("<textarea class='baseComment' data-comment-namespace='{mclassdef.mmodule.full_name}::{name}' data-comment-location='{nclass.doc_location.to_s}'>{nclass.full_comment}</textarea>")
+                                               if nclass.full_comment == "" then
+                                                       page.append("<p class='info inheritance'>")
+                                                       page.append("<span class=\"noComment\">no comment for </span>")
+                                               else
+                                                       page.append("<div class='comment'>{nclass.full_markdown}</div>")
+                                                       page.append("<p class='info inheritance'>")
+                                               end
+                                               if mclassdef.is_intro then
+                                                       page.append("introduction in ")
+                                               else
+                                                       page.append("refinement in ")
+                                               end
+                                               mclassdef.mmodule.html_full_namespace(page)
+                                               page.append(" {page.show_source(nclass.location)}</p>")
+                                       end
+                               end
+                       end
+               else
+                       # comments for intro
+                       if page.ctx.mbuilder.mclassdef2nclassdef.has_key(intro) then
+                               var nclass = page.ctx.mbuilder.mclassdef2nclassdef[intro]
                                if nclass isa AStdClassdef then
+                                       page.append("<textarea class='baseComment' data-comment-namespace='{intro.mmodule.full_name}::{name}' data-comment-location='{nclass.doc_location.to_s}'>{nclass.full_comment}</textarea>")
                                        if nclass.full_comment == "" then
                                                page.append("<p class='info inheritance'>")
                                                page.append("<span class=\"noComment\">no comment for </span>")
                                        else
-                                               page.append("<pre>{nclass.full_comment}</pre>")
+                                               page.append("<div class='comment'>{nclass.full_markdown}</div>")
                                                page.append("<p class='info inheritance'>")
                                        end
-                                       if mclassdef.is_intro then
-                                               page.append("introduction in ")
-                                       else
-                                               page.append("refinement in ")
-                                       end
-                                       mclassdef.mmodule.html_full_namespace(page)
+                                       page.append("introduction in ")
+                                       intro.mmodule.html_full_namespace(page)
                                        page.append(" {page.show_source(nclass.location)}</p>")
                                end
                        end
                end
                page.append("</div>")
        end
-       
+
        private fun html_redefs(page: NitdocModule) do
                page.mmodule.linearize_mclassdefs(mclassdefs)
                page.append("<div class='refinements'>")
@@ -1413,21 +1471,23 @@ redef class MClass
 end
 
 redef class MProperty
+       # Escape name for html output
+       private fun html_name: String do return name.html_escape
+
        # Return the property namespace decorated with html
-       fun html_namespace(page: NitdocPage) do
+       #       <span>intro_module::intro_class::html_link</span>
+       private fun html_namespace(page: NitdocPage) do
                intro_mclassdef.mclass.html_namespace(page)
                page.append(intro_mclassdef.mclass.html_short_signature)
                page.append("::<span>")
                intro.html_link(page)
                page.append("</span>")
        end
-
-       # Escape name for html output
-       redef fun name do return super.html_escape
 end
 
 redef class MType
-       fun html_link(page: NitdocPage) is abstract
+       # Link to the type definition in the nitdoc page
+       private fun html_link(page: NitdocPage) is abstract
 end
 
 redef class MClassType
@@ -1443,7 +1503,7 @@ end
 
 redef class MGenericType
        redef fun html_link(page) do
-               page.append("<a href='{mclass.url}'>{mclass.name}</a>[")
+               page.append("<a href='{mclass.url}'>{mclass.html_name}</a>[")
                for i in [0..arguments.length[ do
                        arguments[i].html_link(page)
                        if i < arguments.length - 1 then page.append(", ")
@@ -1465,7 +1525,7 @@ end
 
 redef class MClassDef
        # Return the classdef namespace decorated with html
-       fun html_namespace(page: NitdocPage) do
+       private fun html_namespace(page: NitdocPage) do
                mmodule.html_full_namespace(page)
                page.append("::<span>")
                mclass.html_link(page)
@@ -1474,7 +1534,15 @@ redef class MClassDef
 end
 
 redef class MPropDef
-       fun url: String do
+       # Return the full qualified name of the mpropdef
+       #       module::classdef::name
+       private fun full_name: String do
+               return "{mclassdef.mclass.public_owner.name}::{mclassdef.mclass.name}::{mproperty.name}"
+       end
+
+       # URL into the nitdoc page
+       #       class_owner_name.html#anchor
+       private fun url: String do
                if url_cache == null then
                        url_cache = "{mclassdef.mclass.url}#{anchor}"
                end
@@ -1482,23 +1550,26 @@ redef class MPropDef
        end
        private var url_cache: nullable String
 
-       fun anchor: String do
+       # html anchor id for the property in a nitdoc class page
+       #       PROP_mclass_propertyname
+       private fun anchor: String do
                if anchor_cache == null then
-                       anchor_cache = "PROP_{mclassdef.mclass.public_owner.name}_{mproperty.name}"
+                       anchor_cache = "PROP_{mclassdef.mclass.public_owner.name}_{mproperty.name.replace(" ", "_")}"
                end
                return anchor_cache.as(not null)
        end
        private var anchor_cache: nullable String
 
-       # Return a link (html a tag) to the nitdoc class page
-       fun html_link(page: NitdocPage) do
+       # Return a link to property into the nitdoc class page
+       #       <a href="url" title="short_comment">html_name</a>
+       private fun html_link(page: NitdocPage) do
                if html_link_cache == null then
                        var res = new Buffer
                        if page.ctx.mbuilder.mpropdef2npropdef.has_key(self) then
                                var nprop = page.ctx.mbuilder.mpropdef2npropdef[self]
-                               res.append("<a href=\"{url}\" title=\"{nprop.short_comment}\">{mproperty.name}</a>")
+                               res.append("<a href=\"{url}\" title=\"{nprop.short_comment}\">{mproperty.html_name}</a>")
                        else
-                               res.append("<a href=\"{url}\">{mproperty.name}</a>")
+                               res.append("<a href=\"{url}\">{mproperty.html_name}</a>")
                        end
                        html_link_cache = res.to_s
                end
@@ -1507,6 +1578,7 @@ redef class MPropDef
        private var html_link_cache: nullable String
 
        # Return a list item for the mpropdef
+       #       <li>html_link</li>
        private fun html_sidebar_item(page: NitdocClass) do
                if is_intro and mclassdef.mclass == page.mclass then
                        page.append("<li class='intro'>")
@@ -1525,20 +1597,17 @@ redef class MPropDef
        private fun html_full_desc(page: NitdocPage, ctx: MClass) is abstract
        private fun html_info(page: NitdocPage, ctx: MClass) is abstract
 
-       fun full_name: String do
-               return "{mclassdef.mclass.public_owner.name}::{mclassdef.mclass.name}::{mproperty.name}"
-       end
-
        private fun html_comment(page: NitdocPage) do
                page.append("<div class='description'>")
                if not is_intro then
                        if page.ctx.mbuilder.mpropdef2npropdef.has_key(mproperty.intro) then
                                var intro_nprop = page.ctx.mbuilder.mpropdef2npropdef[mproperty.intro]
+                               page.append("<textarea class='baseComment' data-comment-namespace='{mproperty.intro.mclassdef.mmodule.full_name}::{mproperty.intro.mclassdef.mclass.name}::{mproperty.name}' data-comment-location='{intro_nprop.doc_location.to_s}'>{intro_nprop.full_comment}</textarea>")
                                if intro_nprop.full_comment.is_empty then
                                        page.append("<p class='info inheritance'>")
                                        page.append("<span class=\"noComment\">no comment for </span>")
                                else
-                                       page.append("<pre>{intro_nprop.full_comment}</pre>")
+                                       page.append("<div class='comment'>{intro_nprop.full_markdown}</div>")
                                        page.append("<p class='info inheritance'>")
                                end
                                page.append("introduction in ")
@@ -1548,16 +1617,17 @@ redef class MPropDef
                end
                if page.ctx.mbuilder.mpropdef2npropdef.has_key(self) then
                        var nprop = page.ctx.mbuilder.mpropdef2npropdef[self]
+                       page.append("<textarea class='baseComment' data-comment-namespace='{mclassdef.mmodule.full_name}::{mclassdef.mclass.name}::{mproperty.name}' data-comment-location='{nprop.doc_location.to_s}'>{nprop.full_comment}</textarea>")
                        if nprop.full_comment == "" then
                                page.append("<p class='info inheritance'>")
                                page.append("<span class=\"noComment\">no comment for </span>")
                        else
-                               page.append("<pre>{nprop.full_comment}</pre>")
+                               page.append("<div class='comment'>{nprop.full_markdown}</div>")
                                page.append("<p class='info inheritance'>")
                        end
                        if is_intro then
                                page.append("introduction in ")
-                       else 
+                       else
                                page.append("redefinition in ")
                        end
                        mclassdef.html_namespace(page)
@@ -1581,7 +1651,7 @@ redef class MMethodDef
                page.append("<article class='{classes.join(" ")}' id='{anchor}'>")
                if page.ctx.mbuilder.mpropdef2npropdef.has_key(self) then
                        page.append("<h3 class='signature' data-untyped-signature='{mproperty.name}{msignature.untyped_signature(page)}'>")
-                       page.append("<span>{mproperty.name}")
+                       page.append("<span>{mproperty.html_name}")
                        msignature.html_signature(page)
                        page.append("</span></h3>")
                else
@@ -1594,7 +1664,7 @@ redef class MMethodDef
                html_comment(page)
                page.append("</article>")
        end
-       
+
        redef fun html_info(page, ctx) do
                page.append("<div class='info'>")
                if mproperty.visibility < public_visibility then page.append("{mproperty.visibility.to_s} ")
@@ -1617,7 +1687,7 @@ redef class MVirtualTypeDef
                if is_redef then classes.add("redef")
                classes.add(mproperty.visibility.to_s)
                page.append("<article class='{classes.join(" ")}' id='{anchor}'>")
-               page.append("<h3 class='signature' data-untyped-signature='{mproperty.name}'><span>{mproperty.name}: ")
+               page.append("<h3 class='signature' data-untyped-signature='{mproperty.name}'><span>{mproperty.html_name}: ")
                bound.html_link(page)
                page.append("</span></h3>")
                html_info(page, ctx)
@@ -1676,54 +1746,97 @@ end
 # Nodes redefs
 #
 
+redef class ADoc
+       private fun short_comment: String do
+               return n_comment.first.text.substring_from(2).replace("\n", "").html_escape
+       end
+
+       private fun full_comment: String do
+               var res = new Buffer
+               for t in n_comment do
+                       var text = t.text
+                       text = text.substring_from(1)
+                       if text.first == ' ' then text = text.substring_from(1)
+                       res.append(text.html_escape)
+               end
+               return res.to_s
+       end
+end
+
 redef class AModule
        private fun short_comment: String do
                if n_moduledecl != null and n_moduledecl.n_doc != null then
-                       return n_moduledecl.n_doc.n_comment.first.text.substring_from(2).replace("\n", "").html_escape
+                       return n_moduledecl.n_doc.short_comment
                end
                return ""
        end
 
        private fun full_comment: String do
-               var res = new Buffer
                if n_moduledecl != null and n_moduledecl.n_doc != null then
-                       for t in n_moduledecl.n_doc.n_comment do
-                               res.append(t.text.substring_from(1).html_escape)
-                       end
+                       return n_moduledecl.n_doc.full_comment
                end
-               return res.to_s
+               return ""
+       end
+
+       private fun full_markdown: String do
+               if n_moduledecl != null and n_moduledecl.n_doc != null then
+                       return n_moduledecl.n_doc.full_markdown.html
+               end
+               return ""
+       end
+
+       private fun doc_location: Location do
+               if n_moduledecl != null and n_moduledecl.n_doc != null then
+                       return n_moduledecl.n_doc.location
+               end
+               return location
        end
 end
 
 redef class AStdClassdef
        private fun short_comment: String do
-               if n_doc != null then return n_doc.n_comment.first.text.substring_from(2).replace("\n", "").html_escape
+               if n_doc != null then return n_doc.short_comment
                return ""
        end
 
        private fun full_comment: String do
-               var res = new Buffer
-               if n_doc != null then
-                       for t in n_doc.n_comment do res.append(t.text.substring_from(1).html_escape)
-               end
-               return res.to_s
+               if n_doc != null then return n_doc.full_comment
+               return ""
+       end
+
+       private fun full_markdown: String do
+               if n_doc != null then return n_doc.full_markdown.html
+               return ""
+       end
+
+       private fun doc_location: Location do
+               if n_doc != null then return n_doc.location
+               return location
        end
 end
 
 redef class APropdef
        private fun short_comment: String do
-               if n_doc != null then return n_doc.n_comment.first.text.substring_from(2).replace("\n", "").html_escape
+               if n_doc != null then return n_doc.short_comment
                return ""
        end
 
        private fun full_comment: String do
-               var res = new Buffer
-               if n_doc != null then
-                       for t in n_doc.n_comment do res.append(t.text.substring_from(1).html_escape)
-               end
-               return res.to_s
+               if n_doc != null then return n_doc.full_comment
+               return ""
+       end
+
+       private fun full_markdown: String do
+               if n_doc != null then return n_doc.full_markdown.html
+               return ""
+       end
+
+       private fun doc_location: Location do
+               if n_doc != null then return n_doc.location
+               return location
        end
 end
 
+
 var nitdoc = new NitdocContext
 nitdoc.generate_nitdoc