remove all non-word characters from heading id's. see markedjs#181.

zergeborg · Aug 4, 2013 · 25a4531 · 25a4531
1 parent 327f07b
commit 25a4531
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/lib/marked.js b/lib/marked.js
@@ -838,7 +838,7 @@ Parser.prototype.tok = function() {
       return '<h'
         + this.token.depth
         + ' id="'
-        + this.token.text.toLowerCase().replace(/\s/g, '-')
+        + this.token.text.toLowerCase().replace(/[^\w]+/g, '-')
         + '">'
         + this.inline.output(this.token.text)
         + '</h'

diff --git a/test/index.js b/test/index.js
@@ -349,7 +349,7 @@ function fix(options) {
           .replace(/&lt;/g, '<')
           .replace(/&amp;/g, '&');
 
-        id = id.toLowerCase().replace(/\s/g, '-');
+        id = id.toLowerCase().replace(/[^\w]+/g, '-');
 
         return '<' + h + ' id="' + id + '">' + text + '</' + h + '>';
       });

diff --git a/test/tests/markdown_documentation_basics.html b/test/tests/markdown_documentation_basics.html
@@ -1,4 +1,4 @@
-<h1 id="markdown:-basics">Markdown: Basics</h1>
+<h1 id="markdown-basics">Markdown: Basics</h1>
 
 <ul id="ProjectSubmenu">
     <li><a href="/projects/markdown/" title="Markdown Project Page">Main</a></li>
@@ -8,7 +8,7 @@ <h1 id="markdown:-basics">Markdown: Basics</h1>
     <li><a href="/projects/markdown/dingus" title="Online Markdown Web Form">Dingus</a></li>
 </ul>
 
-<h2 id="getting-the-gist-of-markdown's-formatting-syntax">Getting the Gist of Markdown&#39;s Formatting Syntax</h2>
+<h2 id="getting-the-gist-of-markdown-s-formatting-syntax">Getting the Gist of Markdown&#39;s Formatting Syntax</h2>
 
 <p>This page offers a brief overview of what it&#39;s like to use Markdown.
 The <a href="/projects/markdown/syntax" title="Markdown Syntax">syntax page</a> provides complete, detailed documentation for
@@ -24,7 +24,7 @@ <h2 id="getting-the-gist-of-markdown's-formatting-syntax">Getting the Gist of Ma
 <p><strong>Note:</strong> This document is itself written using Markdown; you
 can <a href="/projects/markdown/basics.text">see the source for it by adding &#39;.text&#39; to the URL</a>.</p>
 
-<h2 id="paragraphs,-headers,-blockquotes">Paragraphs, Headers, Blockquotes</h2>
+<h2 id="paragraphs-headers-blockquotes">Paragraphs, Headers, Blockquotes</h2>
 
 <p>A paragraph is simply one or more consecutive lines of text, separated
 by one or more blank lines. (A blank line is any line that looks like a

diff --git a/test/tests/markdown_documentation_syntax.html b/test/tests/markdown_documentation_syntax.html
@@ -1,4 +1,4 @@
-<h1 id="markdown:-syntax">Markdown: Syntax</h1>
+<h1 id="markdown-syntax">Markdown: Syntax</h1>
 
 <ul id="ProjectSubmenu">
     <li><a href="/projects/markdown/" title="Markdown Project Page">Main</a></li>