markdown-it - npm Package Compare versions

Comparing version 4.3.1 to 4.4.0

CHANGELOG.md

@@ -0,1 +1,8 @@
+4.4.0 / 2015-07-18
+------------------
+
+- Updated HTML blocks logic to CM 0.21 spec.
+- Minor fixes.
+
+
 4.3.1 / 2015-07-15
@@ -2,0 +9,0 @@ ------------------

lib/common/html_blocks.js

@@ -6,19 +6,22 @@ // List of valid html blocks names, accorting to commonmark spec
 
-var html_blocks = {};
 
-[
+module.exports = [
+  'address',
   'article',
   'aside',
-  'button',
+  'base',
+  'basefont',
   'blockquote',
   'body',
-  'canvas',
   'caption',
+  'center',
   'col',
   'colgroup',
   'dd',
+  'details',
+  'dialog',
+  'dir',
   'div',
   'dl',
   'dt',
-  'embed',
   'fieldset',
@@ -29,36 +32,38 @@ 'figcaption',
   'form',
+  'frame',
+  'frameset',
   'h1',
-  'h2',
-  'h3',
-  'h4',
-  'h5',
-  'h6',
+  'head',
   'header',
-  'hgroup',
   'hr',
-  'iframe',
+  'html',
+  'legend',
   'li',
-  'map',
-  'object',
+  'link',
+  'main',
+  'menu',
+  'menuitem',
+  'meta',
+  'nav',
+  'noframes',
   'ol',
-  'output',
+  'optgroup',
+  'option',
   'p',
+  'param',
   'pre',
-  'progress',
-  'script',
   'section',
-  'style',
+  'source',
+  'title',
+  'summary',
   'table',
   'tbody',
   'td',
-  'textarea',
   'tfoot',
   'th',
+  'thead',
+  'title',
   'tr',
-  'thead',
-  'ul',
-  'video'
-].forEach(function (name) { html_blocks[name] = true; });
-
-
-module.exports = html_blocks;
+  'track',
+  'ul'
+];

lib/common/html_re.js

@@ -25,3 +25,5 @@ // Regexps to match html elements
                         '|' + processing + '|' + declaration + '|' + cdata + ')');
+var HTML_OPEN_CLOSE_TAG_RE = new RegExp('^(?:' + open_tag + '|' + close_tag + ')');
 
 module.exports.HTML_TAG_RE = HTML_TAG_RE;
+module.exports.HTML_OPEN_CLOSE_TAG_RE = HTML_OPEN_CLOSE_TAG_RE;

lib/rules_block/html_block.js

@@ -7,58 +7,57 @@ // HTML block
 var block_names = require('../common/html_blocks');
+var HTML_OPEN_CLOSE_TAG_RE = require('../common/html_re').HTML_OPEN_CLOSE_TAG_RE;
 
+// An array of opening and corresponding closing sequences for html tags,
+// last argument defines whether it can terminate a paragraph or not
+//
+var HTML_SEQUENCES = [
+  [ /^<(script|pre|style)(?=(\s|>|$))/i, /<\/(script|pre|style)>/i, true ],
+  [ /^<!--/,        /-->/,   true ],
+  [ /^<\?/,         /\?>/,   true ],
+  [ /^<![A-Z]/,     />/,     true ],
+  [ /^<!\[CDATA\[/, /\]\]>/, true ],
+  [ new RegExp('^</?(' + block_names.join('|') + ')(?=(\\s|/?>|$))', 'i'), /^$/, true ],
+  [ new RegExp(HTML_OPEN_CLOSE_TAG_RE.source + '\\s*$'),  /^$/, false ]
+];
 
-var HTML_TAG_OPEN_RE = /^<([a-zA-Z][a-zA-Z0-9]{0,14})[\s\/>]/;
-var HTML_TAG_CLOSE_RE = /^<\/([a-zA-Z][a-zA-Z0-9]{0,14})[\s>]/;
 
-function isLetter(ch) {
-  /*eslint no-bitwise:0*/
-  var lc = ch | 0x20; // to lower case
-  return (lc >= 0x61/* a */) && (lc <= 0x7a/* z */);
-}
-
 module.exports = function html_block(state, startLine, endLine, silent) {
-  var ch, match, nextLine, token,
-      pos = state.bMarks[startLine],
-      max = state.eMarks[startLine],
-      shift = state.tShift[startLine];
+  var i, nextLine, token, lineText,
+      pos = state.bMarks[startLine] + state.tShift[startLine],
+      max = state.eMarks[startLine];
 
-  pos += shift;
-
   if (!state.md.options.html) { return false; }
 
-  if (shift > 3 || pos + 2 >= max) { return false; }
-
   if (state.src.charCodeAt(pos) !== 0x3C/* < */) { return false; }
 
-  ch = state.src.charCodeAt(pos + 1);
+  lineText = state.src.slice(pos, max);
 
-  if (ch === 0x21/* ! */ || ch === 0x3F/* ? */) {
-    // Directive start / comment start / processing instruction start
-    if (silent) { return true; }
+  for (i = 0; i < HTML_SEQUENCES.length; i++) {
+    if (HTML_SEQUENCES[i][0].test(lineText)) { break; }
+  }
 
-  } else if (ch === 0x2F/* / */ || isLetter(ch)) {
+  if (i === HTML_SEQUENCES.length) { return false; }
 
-    // Probably start or end of tag
-    if (ch === 0x2F/* \ */) {
-      // closing tag
-      match = state.src.slice(pos, max).match(HTML_TAG_CLOSE_RE);
-      if (!match) { return false; }
-    } else {
-      // opening tag
-      match = state.src.slice(pos, max).match(HTML_TAG_OPEN_RE);
-      if (!match) { return false; }
-    }
-    // Make sure tag name is valid
-    if (block_names[match[1].toLowerCase()] !== true) { return false; }
-    if (silent) { return true; }
-
-  } else {
-    return false;
+  if (silent) {
+    // true if this sequence can be a terminator, false otherwise
+    return HTML_SEQUENCES[i][2];
   }
 
+  nextLine = startLine + 1;
+
   // If we are here - we detected HTML block.
-  // Let's roll down till empty line (block end).
-  nextLine = startLine + 1;
-  while (nextLine < state.lineMax && !state.isEmpty(nextLine)) {
-    nextLine++;
+  // Let's roll down till block end.
+  if (!HTML_SEQUENCES[i][1].test(lineText)) {
+    for (; nextLine < endLine; nextLine++) {
+      if (state.tShift[nextLine] < state.blkIndent) { break; }
+
+      pos = state.bMarks[nextLine] + state.tShift[nextLine];
+      max = state.eMarks[nextLine];
+      lineText = state.src.slice(pos, max);
+
+      if (HTML_SEQUENCES[i][1].test(lineText)) {
+        if (lineText.length !== 0) { nextLine++; }
+        break;
+      }
+    }
   }
@@ -69,6 +68,6 @@
   token         = state.push('html_block', '', 0);
-  token.map     = [ startLine, state.line ];
-  token.content = state.getLines(startLine, nextLine, 0, true);
+  token.map     = [ startLine, nextLine ];
+  token.content = state.getLines(startLine, nextLine, state.blkIndent, true);
 
   return true;
 };

lib/rules_block/list.js

@@ -34,3 +34,4 @@ // Lists
   var ch,
-      pos = state.bMarks[startLine] + state.tShift[startLine],
+      start = state.bMarks[startLine] + state.tShift[startLine],
+      pos = start,
       max = state.eMarks[startLine];
@@ -52,2 +53,7 @@
     if (ch >= 0x30/* 0 */ && ch <= 0x39/* 9 */) {
+
+      // List marker should have no more than 9 digits
+      // (prevents integer overflow in browsers)
+      if (pos - start >= 10) { return -1; }
+
       continue;
@@ -54,0 +60,0 @@ }

lib/rules_block/reference.js

@@ -148,2 +148,13 @@ 'use strict';
   if (pos < max && str.charCodeAt(pos) !== 0x0A) {
+    if (title) {
+      // garbage at the end of the line after title,
+      // but it could still be a valid reference if we roll back
+      title = '';
+      pos = destEndPos;
+      lines = destEndLineNo;
+      while (pos < max && str.charCodeAt(pos) === 0x20/* space */) { pos++; }
+    }
+  }
+
+  if (pos < max && str.charCodeAt(pos) !== 0x0A) {
     // garbage at the end of the line
@@ -150,0 +161,0 @@ return false;

package.json

 {
   "name": "markdown-it",
-  "version": "4.3.1",
+  "version": "4.4.0",
   "description": "Markdown-it - modern pluggable markdown parser.",
@@ -5,0 +5,0 @@ "keywords": [

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

502074

increased by0.46%

Lines of code

11000

increased by0.42%

No dependency changes