From 5e6a04a837d9b2949677f06c89f9ff47087926e2 Mon Sep 17 00:00:00 2001
From: ota-meshi <otameshiyo23@gmail.com>
Date: Thu, 29 Mar 2018 19:24:29 +0900
Subject: [PATCH 1/6] [New] Add `vue/html-content-newline`

---
 lib/rules/html-content-newline.js       | 127 +++++++++++++++
 tests/lib/rules/html-content-newline.js | 197 ++++++++++++++++++++++++
 2 files changed, 324 insertions(+)
 create mode 100644 lib/rules/html-content-newline.js
 create mode 100644 tests/lib/rules/html-content-newline.js

diff --git a/lib/rules/html-content-newline.js b/lib/rules/html-content-newline.js
new file mode 100644
index 000000000..9fbc128e8
--- /dev/null
+++ b/lib/rules/html-content-newline.js
@@ -0,0 +1,127 @@
+/**
+ * @author Yosuke Ota
+ * See LICENSE file in root directory for full license.
+ */
+'use strict'
+
+// ------------------------------------------------------------------------------
+// Requirements
+// ------------------------------------------------------------------------------
+
+const utils = require('../utils')
+
+// ------------------------------------------------------------------------------
+// Helpers
+// ------------------------------------------------------------------------------
+
+function parseOptions (options) {
+  return Object.assign({
+    'singleline': 'ignore',
+    'multiline': 'always',
+    'ignoreNames': ['pre', 'textarea'],
+
+    detectType (node) {
+      if (node.startTag.loc.start.line === node.startTag.loc.end.line &&
+        node.endTag.loc.start.line === node.endTag.loc.end.line) {
+        return this.singleline
+      }
+      return this.multiline
+    }
+  }, options)
+}
+
+function getBreakPhrase (breaks) {
+  return breaks ? 'line break' : 'no line breaks'
+}
+
+// ------------------------------------------------------------------------------
+// Rule Definition
+// ------------------------------------------------------------------------------
+
+module.exports = {
+  meta: {
+    docs: {
+      description: 'require or disallow a line break before and after html contents',
+      category: undefined,
+      url: 'https://github.com/vuejs/eslint-plugin-vue/blob/v4.4.0/docs/rules/html-content-newline.md'
+    },
+    fixable: 'whitespace',
+    schema: [{
+      type: 'object',
+      properties: {
+        'singleline': { enum: ['ignore', 'always', 'never'] },
+        'multiline': { enum: ['ignore', 'always', 'never'] },
+        'ignoreNames': {
+          type: 'array',
+          items: { type: 'string' },
+          uniqueItems: true,
+          additionalItems: false
+        }
+      },
+      additionalProperties: false
+    }]
+  },
+
+  create (context) {
+    const options = parseOptions(context.options[0])
+    const template = context.parserServices.getTemplateBodyTokenStore && context.parserServices.getTemplateBodyTokenStore()
+
+    return utils.defineTemplateBodyVisitor(context, {
+      'VElement' (node) {
+        if (node.startTag.selfClosing || !node.endTag) {
+          // self closing
+          return
+        }
+        if (options.ignoreNames.indexOf(node.name) >= 0) {
+          return
+        }
+        const type = options.detectType(node)
+        if (type === 'ignore') {
+          return
+        }
+        debugger// TODO
+        const getTokenOption = { includeComments: true, filter: (token) => token.type !== 'HTMLWhitespace' }
+        const contentFirst = template.getTokenAfter(node.startTag, getTokenOption)
+        const contentLast = template.getTokenBefore(node.endTag, getTokenOption)
+        const hasBeforeBreaks = node.startTag.loc.end.line < contentFirst.loc.start.line
+        const hasAfterBreaks = contentLast.loc.end.line < node.endTag.loc.start.line
+        const needBreaks = type === 'always'
+        if (needBreaks !== hasBeforeBreaks) {
+          context.report({
+            node: template.getLastToken(node.startTag),
+            loc: {
+              start: node.startTag.loc.end,
+              end: contentFirst.loc.start
+            },
+            message: 'Expected {{expected}} after closing bracket, but {{actual}} found.',
+            data: {
+              expected: getBreakPhrase(needBreaks),
+              actual: getBreakPhrase(hasBeforeBreaks)
+            },
+            fix: type === 'always'
+              ? (fixer) => fixer.insertTextAfter(node.startTag, '\n')
+              : (fixer) => fixer.removeRange([node.startTag.range[1] + 1, contentFirst.range[0]])
+          })
+        }
+
+        if (needBreaks !== hasAfterBreaks) {
+          context.report({
+            node: template.getFirstToken(node.endTag),
+            loc: {
+              start: contentLast.loc.end,
+              end: node.endTag.loc.start
+            },
+            message: 'Expected {{expected}} before open bracket, but {{actual}} found.',
+            data: {
+              expected: getBreakPhrase(needBreaks),
+              actual: getBreakPhrase(hasAfterBreaks)
+            },
+            fix: type === 'always'
+              ? (fixer) => fixer.insertTextBefore(node.endTag, '\n')
+              : (fixer) => fixer.removeRange([contentLast.range[1] + 1, node.endTag.range[0]])
+          })
+        }
+      }
+    })
+  }
+}
diff --git a/tests/lib/rules/html-content-newline.js b/tests/lib/rules/html-content-newline.js
new file mode 100644
index 000000000..7a7d1fe40
--- /dev/null
+++ b/tests/lib/rules/html-content-newline.js
@@ -0,0 +1,197 @@
+/**
+ * @author Yosuke Ota
+ */
+'use strict'
+
+// ------------------------------------------------------------------------------
+// Requirements
+// ------------------------------------------------------------------------------
+
+const rule = require('../../../lib/rules/html-content-newline')
+const RuleTester = require('eslint').RuleTester
+
+// ------------------------------------------------------------------------------
+// Tests
+// ------------------------------------------------------------------------------
+
+const tester = new RuleTester({
+  parser: 'vue-eslint-parser',
+  parserOptions: {
+    ecmaVersion: 2015
+  }
+})
+
+tester.run('html-content-newline', rule, {
+  valid: [
+    `<template><div class="panel">content</div></template>`,
+    `
+      <template>
+        <div class="panel">
+          content
+        </div>
+      </template>`,
+    `
+      <template>
+        <div
+          class="panel"
+        >
+          content
+        </div>
+      </template>`,
+    {
+      code: `
+        <template>
+          <div class="panel">
+            content
+          </div>
+        </template>`,
+      options: [{
+        singleline: 'always',
+        multiline: 'never'
+      }]
+    },
+    {
+      code: `
+        <template>
+          <div
+            class="panel"
+          >content</div>
+        </template>`,
+      options: [{
+        singleline: 'always',
+        multiline: 'never'
+      }]
+    },
+    {
+      code: `
+        <template>
+          <self-closing />
+        </template>`,
+      options: [{
+        singleline: 'always',
+        multiline: 'never'
+      }]
+    },
+    // Ignore if no closing brackets
+    `
+      <template>
+        <div
+          id=
+          ""
+    `
+  ],
+  invalid: [
+    {
+      code: `
+        <template>
+          <div
+            class="panel"
+          >content</div>
+        </template>
+      `,
+      output: `
+        <template>
+          <div
+            class="panel"
+          >
+content
+</div>
+        </template>
+      `,
+      errors: [
+        {
+          message: 'Expected line break after closing bracket, but no line breaks found.',
+          line: 5,
+          column: 12,
+          nodeType: 'HTMLTagClose',
+          endLine: 5,
+          endColumn: 12
+        },
+        {
+          message: 'Expected line break before open bracket, but no line breaks found.',
+          line: 5,
+          column: 19,
+          nodeType: 'HTMLEndTagOpen',
+          endLine: 5,
+          endColumn: 19
+        }
+      ]
+    },
+    {
+      code: `
+        <template>
+          <div class="panel">panel</div>
+        </template>
+      `,
+      options: [{
+        singleline: 'always',
+        multiline: 'never'
+      }],
+      output: `
+        <template>
+          <div class="panel">
+panel
+</div>
+        </template>
+      `,
+      errors: [
+        {
+          message: 'Expected line break after closing bracket, but no line breaks found.',
+          line: 3,
+          column: 30,
+          nodeType: 'HTMLTagClose',
+          endLine: 3,
+          endColumn: 30
+        },
+        {
+          message: 'Expected line break before open bracket, but no line breaks found.',
+          line: 3,
+          column: 35,
+          nodeType: 'HTMLEndTagOpen',
+          endLine: 3,
+          endColumn: 35
+        }]
+    },
+    {
+      code: `
+        <template>
+          <div
+            class="panel"
+          >
+            content
+          </div>
+        </template>
+      `,
+      options: [{
+        singleline: 'always',
+        multiline: 'never'
+      }],
+      output: `
+        <template>
+          <div
+            class="panel"
+          >
+content
+</div>
+        </template>
+      `,
+      errors: [{
+        message: 'Expected no line breaks after closing bracket, but line break found.',
+        line: 5,
+        column: 12,
+        nodeType: 'HTMLTagClose',
+        endLine: 6,
+        endColumn: 13
+      },
+      {
+        message: 'Expected no line breaks before open bracket, but line break found.',
+        line: 6,
+        column: 20,
+        nodeType: 'HTMLEndTagOpen',
+        endLine: 7,
+        endColumn: 11
+      }
+      ]
+    }
+  ]
+})

From f4aa12e66e40b6af16721926b5a1088c9dc7bb0c Mon Sep 17 00:00:00 2001
From: ota-meshi <otameshiyo23@gmail.com>
Date: Thu, 29 Mar 2018 21:02:04 +0900
Subject: [PATCH 2/6] [update] add test case

---
 lib/rules/html-content-newline.js       |  26 +-
 tests/lib/rules/html-content-newline.js | 347 ++++++++++++++++++++++--
 2 files changed, 343 insertions(+), 30 deletions(-)

diff --git a/lib/rules/html-content-newline.js b/lib/rules/html-content-newline.js
index 9fbc128e8..0fa23c937 100644
--- a/lib/rules/html-content-newline.js
+++ b/lib/rules/html-content-newline.js
@@ -18,9 +18,13 @@ function parseOptions (options) {
   return Object.assign({
     'singleline': 'ignore',
     'multiline': 'always',
+    'hasChildElements': null,
     'ignoreNames': ['pre', 'textarea'],
 
     detectType (node) {
+      if (this.hasChildElements && node.children.filter(c => c.type === 'VElement').length) {
+        return this.hasChildElements
+      }
       if (node.startTag.loc.start.line === node.startTag.loc.end.line &&
         node.endTag.loc.start.line === node.endTag.loc.end.line) {
         return this.singleline
@@ -51,6 +55,7 @@ module.exports = {
       properties: {
         'singleline': { enum: ['ignore', 'always', 'never'] },
         'multiline': { enum: ['ignore', 'always', 'never'] },
+        'hasChildElements': { enum: ['ignore', 'always', 'never'] },
         'ignoreNames': {
           type: 'array',
           items: { type: 'string' },
@@ -72,14 +77,19 @@ module.exports = {
           // self closing
           return
         }
-        if (options.ignoreNames.indexOf(node.name) >= 0) {
-          return
+        let target = node
+        while (target.type === 'VElement') {
+          if (options.ignoreNames.indexOf(target.name) >= 0) {
+          // ignore element name
+            return
+          }
+          target = target.parent
         }
         const type = options.detectType(node)
         if (type === 'ignore') {
+          // 'ignore' option
           return
         }
-        debugger// TODO
         const getTokenOption = { includeComments: true, filter: (token) => token.type !== 'HTMLWhitespace' }
         const contentFirst = template.getTokenAfter(node.startTag, getTokenOption)
         const contentLast = template.getTokenBefore(node.endTag, getTokenOption)
@@ -93,14 +103,15 @@ module.exports = {
               start: node.startTag.loc.end,
               end: contentFirst.loc.start
             },
-            message: 'Expected {{expected}} after closing bracket, but {{actual}} found.',
+            message: `Expected {{expected}} after closing bracket of the "{{name}}" element, but {{actual}} found.`,
             data: {
+              name: node.name,
               expected: getBreakPhrase(needBreaks),
               actual: getBreakPhrase(hasBeforeBreaks)
             },
             fix: type === 'always'
               ? (fixer) => fixer.insertTextAfter(node.startTag, '\n')
-              : (fixer) => fixer.removeRange([node.startTag.range[1] + 1, contentFirst.range[0]])
+              : (fixer) => fixer.removeRange([node.startTag.range[1], contentFirst.range[0]])
           })
         }
 
@@ -111,14 +122,15 @@ module.exports = {
               start: contentLast.loc.end,
               end: node.endTag.loc.start
             },
-            message: 'Expected {{expected}} before open bracket, but {{actual}} found.',
+            message: 'Expected {{expected}} before open bracket of the "{{name}}" element, but {{actual}} found.',
             data: {
+              name: node.name,
               expected: getBreakPhrase(needBreaks),
               actual: getBreakPhrase(hasAfterBreaks)
             },
             fix: type === 'always'
               ? (fixer) => fixer.insertTextBefore(node.endTag, '\n')
-              : (fixer) => fixer.removeRange([contentLast.range[1] + 1, node.endTag.range[0]])
+              : (fixer) => fixer.removeRange([contentLast.range[1], node.endTag.range[0]])
           })
         }
       }
diff --git a/tests/lib/rules/html-content-newline.js b/tests/lib/rules/html-content-newline.js
index 7a7d1fe40..546ab7a55 100644
--- a/tests/lib/rules/html-content-newline.js
+++ b/tests/lib/rules/html-content-newline.js
@@ -62,6 +62,17 @@ tester.run('html-content-newline', rule, {
         multiline: 'never'
       }]
     },
+    // empty
+    `<template><div class="panel"></div></template>`,
+    {
+      code: `<template><div
+      class="panel"></div></template>`,
+      options: [{
+        singleline: 'never',
+        multiline: 'never'
+      }]
+    },
+    // self closing
     {
       code: `
         <template>
@@ -72,6 +83,53 @@ tester.run('html-content-newline', rule, {
         multiline: 'never'
       }]
     },
+    // ignores
+    {
+      code: `
+        <template>
+          <pre>content</pre>
+          <pre
+            id="test-pre"
+          >content</pre>
+          <pre><div>content</div></pre>
+          <textarea>content</textarea>
+          <textarea
+            id="test-textarea"
+          >content</textarea>
+        </template>`,
+      options: [{
+        singleline: 'always',
+        multiline: 'always'
+      }]
+    },
+    {
+      code: `
+        <template>
+          <ignore-tag>content</ignore-tag>
+          <ignore-tag
+            id="test-pre"
+          >content</ignore-tag>
+          <ignore-tag><div>content</div></ignore-tag>
+        </template>`,
+      options: [{
+        singleline: 'always',
+        multiline: 'always',
+        ignoreNames: ['ignore-tag']
+      }]
+    },
+    // hasChildElements
+    {
+      code: `
+        <template>
+          <div><div>
+            content
+          </div></div>
+        </template>`,
+      options: [{
+        singleline: 'always',
+        hasChildElements: 'ignore'
+      }]
+    },
     // Ignore if no closing brackets
     `
       <template>
@@ -100,7 +158,7 @@ content
       `,
       errors: [
         {
-          message: 'Expected line break after closing bracket, but no line breaks found.',
+          message: 'Expected line break after closing bracket of the "div" element, but no line breaks found.',
           line: 5,
           column: 12,
           nodeType: 'HTMLTagClose',
@@ -108,7 +166,7 @@ content
           endColumn: 12
         },
         {
-          message: 'Expected line break before open bracket, but no line breaks found.',
+          message: 'Expected line break before open bracket of the "div" element, but no line breaks found.',
           line: 5,
           column: 19,
           nodeType: 'HTMLEndTagOpen',
@@ -120,7 +178,7 @@ content
     {
       code: `
         <template>
-          <div class="panel">panel</div>
+          <div class="panel">content</div>
         </template>
       `,
       options: [{
@@ -130,13 +188,13 @@ content
       output: `
         <template>
           <div class="panel">
-panel
+content
 </div>
         </template>
       `,
       errors: [
         {
-          message: 'Expected line break after closing bracket, but no line breaks found.',
+          message: 'Expected line break after closing bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 30,
           nodeType: 'HTMLTagClose',
@@ -144,13 +202,14 @@ panel
           endColumn: 30
         },
         {
-          message: 'Expected line break before open bracket, but no line breaks found.',
+          message: 'Expected line break before open bracket of the "div" element, but no line breaks found.',
           line: 3,
-          column: 35,
+          column: 37,
           nodeType: 'HTMLEndTagOpen',
           endLine: 3,
-          endColumn: 35
-        }]
+          endColumn: 37
+        }
+      ]
     },
     {
       code: `
@@ -170,27 +229,269 @@ panel
         <template>
           <div
             class="panel"
-          >
-content
+          >content</div>
+        </template>
+      `,
+      errors: [
+        {
+          message: 'Expected no line breaks after closing bracket of the "div" element, but line break found.',
+          line: 5,
+          column: 12,
+          nodeType: 'HTMLTagClose',
+          endLine: 6,
+          endColumn: 13
+        },
+        {
+          message: 'Expected no line breaks before open bracket of the "div" element, but line break found.',
+          line: 6,
+          column: 20,
+          nodeType: 'HTMLEndTagOpen',
+          endLine: 7,
+          endColumn: 11
+        }
+      ]
+    },
+    // comments
+    {
+      code: `
+        <template>
+          <div><!--comment--></div>
+        </template>
+      `,
+      options: [{
+        singleline: 'always'
+      }],
+      output: `
+        <template>
+          <div>
+<!--comment-->
 </div>
         </template>
       `,
+      errors: [
+        {
+          message: 'Expected line break after closing bracket of the "div" element, but no line breaks found.',
+          line: 3,
+          column: 16
+        },
+        {
+          message: 'Expected line break before open bracket of the "div" element, but no line breaks found.',
+          line: 3,
+          column: 30
+        }
+      ]
+    },
+    {
+      code: `
+        <template>
+          <div>
+          <!--comment-->
+          </div>
+        </template>
+      `,
+      options: [{
+        singleline: 'never'
+      }],
+      output: `
+        <template><div><!--comment--></div></template>
+      `,
       errors: [{
-        message: 'Expected no line breaks after closing bracket, but line break found.',
+        message: 'Expected no line breaks after closing bracket of the "template" element, but line break found.',
+        line: 2,
+        column: 19
+      },
+      {
+        message: 'Expected no line breaks after closing bracket of the "div" element, but line break found.',
+        line: 3,
+        column: 16
+
+      },
+      {
+        message: 'Expected no line breaks before open bracket of the "div" element, but line break found.',
+        line: 4,
+        column: 25
+
+      },
+      {
+        message: 'Expected no line breaks before open bracket of the "template" element, but line break found.',
         line: 5,
-        column: 12,
-        nodeType: 'HTMLTagClose',
-        endLine: 6,
-        endColumn: 13
+        column: 17
+
+      }]
+    },
+    // one error
+    {
+      code: `
+        <template>
+          <div>content
+          </div>
+        </template>
+      `,
+      options: [{
+        singleline: 'always'
+      }],
+      output: `
+        <template>
+          <div>
+content
+          </div>
+        </template>
+      `,
+      errors: [
+        {
+          message: 'Expected line break after closing bracket of the "div" element, but no line breaks found.',
+          line: 3,
+          column: 16
+        }
+      ]
+    },
+    {
+      code: `
+        <template>
+          <div>
+          content</div>
+        </template>
+      `,
+      options: [{
+        singleline: 'always'
+      }],
+      output: `
+        <template>
+          <div>
+          content
+</div>
+        </template>
+      `,
+      errors: [
+        {
+          message: 'Expected line break before open bracket of the "div" element, but no line breaks found.',
+          line: 4,
+          column: 18
+        }
+      ]
+    },
+    {
+      code: `
+        <template><div>content
+          </div></template>
+      `,
+      options: [{
+        singleline: 'never'
+      }],
+      output: `
+        <template><div>content</div></template>
+      `,
+      errors: [{
+        message: 'Expected no line breaks before open bracket of the "div" element, but line break found.',
+        line: 2,
+        column: 31 }]
+    },
+    {
+      code: `
+        <template><div>
+          content</div></template>
+      `,
+      options: [{
+        singleline: 'never'
+      }],
+      output: `
+        <template><div>content</div></template>
+      `,
+      errors: [{
+        message: 'Expected no line breaks after closing bracket of the "div" element, but line break found.',
+        line: 2,
+        column: 24
+      }]
+    },
+    // hasChildElements
+    {
+      code: `
+        <template><div>content<div>content</div>content</div></template>
+      `,
+      options: [{
+        singleline: 'never',
+        hasChildElements: 'always'
+      }],
+      output: `
+        <template>
+<div>
+content<div>content</div>content
+</div>
+</template>
+      `,
+      errors: [{
+        message: 'Expected line break after closing bracket of the "template" element, but no line breaks found.',
+        line: 2,
+        column: 19
+      },
+      {
+        message: 'Expected line break after closing bracket of the "div" element, but no line breaks found.',
+        line: 2,
+        column: 24
+      },
+      {
+        message: 'Expected line break before open bracket of the "div" element, but no line breaks found.',
+        line: 2,
+        column: 56
       },
       {
-        message: 'Expected no line breaks before open bracket, but line break found.',
-        line: 6,
-        column: 20,
-        nodeType: 'HTMLEndTagOpen',
-        endLine: 7,
-        endColumn: 11
-      }
+        message: 'Expected line break before open bracket of the "template" element, but no line breaks found.',
+        line: 2,
+        column: 62
+      }]
+    },
+    // empty
+    {
+      code: `
+        <template>
+          <div></div>
+        </template>
+      `,
+      options: [{
+        singleline: 'always'
+      }],
+      output: `
+        <template>
+          <div>
+</div>
+        </template>
+      `,
+      errors: [
+        {
+          message: 'Expected line break after closing bracket of the "div" element, but no line breaks found.',
+          line: 3,
+          column: 16
+        },
+        {
+          message: 'Expected line break before open bracket of the "div" element, but no line breaks found.',
+          line: 3,
+          column: 16
+        }
+      ]
+    },
+    {
+      code: `
+        <template><div>
+        </div></template>
+      `,
+      options: [{
+        singleline: 'never'
+      }],
+      output: `
+        <template><div></div></template>
+      `,
+      errors: [
+        {
+          message: 'Expected no line breaks after closing bracket of the "div" element, but line break found.',
+          line: 2,
+          column: 24
+        },
+        {
+          message: 'Expected no line breaks before open bracket of the "div" element, but line break found.',
+          line: 2,
+          column: 24
+
+        }
       ]
     }
   ]

From 1c3052ce063799c4f63047703356261207e5bd78 Mon Sep 17 00:00:00 2001
From: ota-meshi <otameshiyo23@gmail.com>
Date: Thu, 29 Mar 2018 21:15:59 +0900
Subject: [PATCH 3/6] [update] multiline break warn

---
 lib/rules/html-content-newline.js       |  42 +++++----
 tests/lib/rules/html-content-newline.js | 112 +++++++++++++++++++-----
 2 files changed, 113 insertions(+), 41 deletions(-)

diff --git a/lib/rules/html-content-newline.js b/lib/rules/html-content-newline.js
index 0fa23c937..ace223272 100644
--- a/lib/rules/html-content-newline.js
+++ b/lib/rules/html-content-newline.js
@@ -34,8 +34,12 @@ function parseOptions (options) {
   }, options)
 }
 
-function getBreakPhrase (breaks) {
-  return breaks ? 'line break' : 'no line breaks'
+function getPhrase (lineBreaks) {
+  switch (lineBreaks) {
+    case 0: return 'no line breaks'
+    case 1: return '1 line break'
+    default: return `${lineBreaks} line breaks`
+  }
 }
 
 // ------------------------------------------------------------------------------
@@ -93,10 +97,10 @@ module.exports = {
         const getTokenOption = { includeComments: true, filter: (token) => token.type !== 'HTMLWhitespace' }
         const contentFirst = template.getTokenAfter(node.startTag, getTokenOption)
         const contentLast = template.getTokenBefore(node.endTag, getTokenOption)
-        const hasBeforeBreaks = node.startTag.loc.end.line < contentFirst.loc.start.line
-        const hasAfterBreaks = contentLast.loc.end.line < node.endTag.loc.start.line
-        const needBreaks = type === 'always'
-        if (needBreaks !== hasBeforeBreaks) {
+        const beforeLineBreaks = contentFirst.loc.start.line - node.startTag.loc.end.line
+        const afterLineBreaks = node.endTag.loc.start.line - contentLast.loc.end.line
+        const expectedLineBreaks = type === 'always' ? 1 : 0
+        if (expectedLineBreaks !== beforeLineBreaks) {
           context.report({
             node: template.getLastToken(node.startTag),
             loc: {
@@ -106,16 +110,18 @@ module.exports = {
             message: `Expected {{expected}} after closing bracket of the "{{name}}" element, but {{actual}} found.`,
             data: {
               name: node.name,
-              expected: getBreakPhrase(needBreaks),
-              actual: getBreakPhrase(hasBeforeBreaks)
+              expected: getPhrase(expectedLineBreaks),
+              actual: getPhrase(beforeLineBreaks)
             },
-            fix: type === 'always'
-              ? (fixer) => fixer.insertTextAfter(node.startTag, '\n')
-              : (fixer) => fixer.removeRange([node.startTag.range[1], contentFirst.range[0]])
+            fix (fixer) {
+              const range = [node.startTag.range[1], contentFirst.range[0]]
+              const text = '\n'.repeat(expectedLineBreaks)
+              return fixer.replaceTextRange(range, text)
+            }
           })
         }
 
-        if (needBreaks !== hasAfterBreaks) {
+        if (expectedLineBreaks !== afterLineBreaks) {
           context.report({
             node: template.getFirstToken(node.endTag),
             loc: {
@@ -125,12 +131,14 @@ module.exports = {
             message: 'Expected {{expected}} before open bracket of the "{{name}}" element, but {{actual}} found.',
             data: {
               name: node.name,
-              expected: getBreakPhrase(needBreaks),
-              actual: getBreakPhrase(hasAfterBreaks)
+              expected: getPhrase(expectedLineBreaks),
+              actual: getPhrase(afterLineBreaks)
             },
-            fix: type === 'always'
-              ? (fixer) => fixer.insertTextBefore(node.endTag, '\n')
-              : (fixer) => fixer.removeRange([contentLast.range[1], node.endTag.range[0]])
+            fix (fixer) {
+              const range = [contentLast.range[1], node.endTag.range[0]]
+              const text = '\n'.repeat(expectedLineBreaks)
+              return fixer.replaceTextRange(range, text)
+            }
           })
         }
       }
diff --git a/tests/lib/rules/html-content-newline.js b/tests/lib/rules/html-content-newline.js
index 546ab7a55..8db90abfa 100644
--- a/tests/lib/rules/html-content-newline.js
+++ b/tests/lib/rules/html-content-newline.js
@@ -158,7 +158,7 @@ content
       `,
       errors: [
         {
-          message: 'Expected line break after closing bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break after closing bracket of the "div" element, but no line breaks found.',
           line: 5,
           column: 12,
           nodeType: 'HTMLTagClose',
@@ -166,7 +166,7 @@ content
           endColumn: 12
         },
         {
-          message: 'Expected line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
           line: 5,
           column: 19,
           nodeType: 'HTMLEndTagOpen',
@@ -194,7 +194,7 @@ content
       `,
       errors: [
         {
-          message: 'Expected line break after closing bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break after closing bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 30,
           nodeType: 'HTMLTagClose',
@@ -202,7 +202,7 @@ content
           endColumn: 30
         },
         {
-          message: 'Expected line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 37,
           nodeType: 'HTMLEndTagOpen',
@@ -234,7 +234,7 @@ content
       `,
       errors: [
         {
-          message: 'Expected no line breaks after closing bracket of the "div" element, but line break found.',
+          message: 'Expected no line breaks after closing bracket of the "div" element, but 1 line break found.',
           line: 5,
           column: 12,
           nodeType: 'HTMLTagClose',
@@ -242,7 +242,7 @@ content
           endColumn: 13
         },
         {
-          message: 'Expected no line breaks before open bracket of the "div" element, but line break found.',
+          message: 'Expected no line breaks before open bracket of the "div" element, but 1 line break found.',
           line: 6,
           column: 20,
           nodeType: 'HTMLEndTagOpen',
@@ -270,12 +270,12 @@ content
       `,
       errors: [
         {
-          message: 'Expected line break after closing bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break after closing bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 16
         },
         {
-          message: 'Expected line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 30
         }
@@ -296,24 +296,24 @@ content
         <template><div><!--comment--></div></template>
       `,
       errors: [{
-        message: 'Expected no line breaks after closing bracket of the "template" element, but line break found.',
+        message: 'Expected no line breaks after closing bracket of the "template" element, but 1 line break found.',
         line: 2,
         column: 19
       },
       {
-        message: 'Expected no line breaks after closing bracket of the "div" element, but line break found.',
+        message: 'Expected no line breaks after closing bracket of the "div" element, but 1 line break found.',
         line: 3,
         column: 16
 
       },
       {
-        message: 'Expected no line breaks before open bracket of the "div" element, but line break found.',
+        message: 'Expected no line breaks before open bracket of the "div" element, but 1 line break found.',
         line: 4,
         column: 25
 
       },
       {
-        message: 'Expected no line breaks before open bracket of the "template" element, but line break found.',
+        message: 'Expected no line breaks before open bracket of the "template" element, but 1 line break found.',
         line: 5,
         column: 17
 
@@ -339,7 +339,7 @@ content
       `,
       errors: [
         {
-          message: 'Expected line break after closing bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break after closing bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 16
         }
@@ -364,7 +364,7 @@ content
       `,
       errors: [
         {
-          message: 'Expected line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
           line: 4,
           column: 18
         }
@@ -382,7 +382,7 @@ content
         <template><div>content</div></template>
       `,
       errors: [{
-        message: 'Expected no line breaks before open bracket of the "div" element, but line break found.',
+        message: 'Expected no line breaks before open bracket of the "div" element, but 1 line break found.',
         line: 2,
         column: 31 }]
     },
@@ -398,7 +398,7 @@ content
         <template><div>content</div></template>
       `,
       errors: [{
-        message: 'Expected no line breaks after closing bracket of the "div" element, but line break found.',
+        message: 'Expected no line breaks after closing bracket of the "div" element, but 1 line break found.',
         line: 2,
         column: 24
       }]
@@ -420,22 +420,22 @@ content<div>content</div>content
 </template>
       `,
       errors: [{
-        message: 'Expected line break after closing bracket of the "template" element, but no line breaks found.',
+        message: 'Expected 1 line break after closing bracket of the "template" element, but no line breaks found.',
         line: 2,
         column: 19
       },
       {
-        message: 'Expected line break after closing bracket of the "div" element, but no line breaks found.',
+        message: 'Expected 1 line break after closing bracket of the "div" element, but no line breaks found.',
         line: 2,
         column: 24
       },
       {
-        message: 'Expected line break before open bracket of the "div" element, but no line breaks found.',
+        message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
         line: 2,
         column: 56
       },
       {
-        message: 'Expected line break before open bracket of the "template" element, but no line breaks found.',
+        message: 'Expected 1 line break before open bracket of the "template" element, but no line breaks found.',
         line: 2,
         column: 62
       }]
@@ -458,12 +458,12 @@ content<div>content</div>content
       `,
       errors: [
         {
-          message: 'Expected line break after closing bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break after closing bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 16
         },
         {
-          message: 'Expected line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 16
         }
@@ -482,17 +482,81 @@ content<div>content</div>content
       `,
       errors: [
         {
-          message: 'Expected no line breaks after closing bracket of the "div" element, but line break found.',
+          message: 'Expected no line breaks after closing bracket of the "div" element, but 1 line break found.',
           line: 2,
           column: 24
         },
         {
-          message: 'Expected no line breaks before open bracket of the "div" element, but line break found.',
+          message: 'Expected no line breaks before open bracket of the "div" element, but 1 line break found.',
           line: 2,
           column: 24
 
         }
       ]
+    },
+    // multi line breaks
+    {
+      code: `
+        <template>
+          <div>
+
+            content
+
+          </div>
+        </template>
+      `,
+      options: [{
+        singleline: 'always'
+      }],
+      output: `
+        <template>
+          <div>
+content
+</div>
+        </template>
+      `,
+      errors: [
+        {
+          message: 'Expected 1 line break after closing bracket of the "div" element, but 2 line breaks found.',
+          line: 3,
+          column: 16
+        },
+        {
+          message: 'Expected 1 line break before open bracket of the "div" element, but 2 line breaks found.',
+          line: 5,
+          column: 20
+        }
+      ]
+    },
+    // mustache
+    {
+      code: `
+        <template>
+          <div>{{content}}</div>
+        </template>
+      `,
+      options: [{
+        singleline: 'always'
+      }],
+      output: `
+        <template>
+          <div>
+{{content}}
+</div>
+        </template>
+      `,
+      errors: [
+        {
+          message: 'Expected 1 line break after closing bracket of the "div" element, but no line breaks found.',
+          line: 3,
+          column: 16
+        },
+        {
+          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
+          line: 3,
+          column: 27
+        }
+      ]
     }
   ]
 })

From d4a82cdfc2d847508e935c9a9bd4b5a4e3bd7c22 Mon Sep 17 00:00:00 2001
From: yosuke ota <otameshiyo23@gmail.com>
Date: Thu, 29 Mar 2018 23:11:56 +0900
Subject: [PATCH 4/6] [update]

---
 lib/rules/html-content-newline.js       |  36 ++---
 tests/lib/rules/html-content-newline.js | 169 ++++++++++++------------
 2 files changed, 104 insertions(+), 101 deletions(-)

diff --git a/lib/rules/html-content-newline.js b/lib/rules/html-content-newline.js
index ace223272..9f5908e5b 100644
--- a/lib/rules/html-content-newline.js
+++ b/lib/rules/html-content-newline.js
@@ -14,23 +14,24 @@ const utils = require('../utils')
 // Helpers
 // ------------------------------------------------------------------------------
 
+function isMultiline (node, contentFirst, contentLast) {
+  if (node.startTag.loc.start.line !== node.startTag.loc.end.line ||
+    node.endTag.loc.start.line !== node.endTag.loc.end.line) {
+    // multiline tag
+    return true
+  }
+  if (contentFirst.loc.start.line < contentLast.loc.end.line) {
+    // multiline contents
+    return true
+  }
+  return false
+}
+
 function parseOptions (options) {
   return Object.assign({
     'singleline': 'ignore',
     'multiline': 'always',
-    'hasChildElements': null,
-    'ignoreNames': ['pre', 'textarea'],
-
-    detectType (node) {
-      if (this.hasChildElements && node.children.filter(c => c.type === 'VElement').length) {
-        return this.hasChildElements
-      }
-      if (node.startTag.loc.start.line === node.startTag.loc.end.line &&
-        node.endTag.loc.start.line === node.endTag.loc.end.line) {
-        return this.singleline
-      }
-      return this.multiline
-    }
+    'ignoreNames': ['pre', 'textarea']
   }, options)
 }
 
@@ -59,7 +60,6 @@ module.exports = {
       properties: {
         'singleline': { enum: ['ignore', 'always', 'never'] },
         'multiline': { enum: ['ignore', 'always', 'never'] },
-        'hasChildElements': { enum: ['ignore', 'always', 'never'] },
         'ignoreNames': {
           type: 'array',
           items: { type: 'string' },
@@ -89,14 +89,14 @@ module.exports = {
           }
           target = target.parent
         }
-        const type = options.detectType(node)
+        const getTokenOption = { includeComments: true, filter: (token) => token.type !== 'HTMLWhitespace' }
+        const contentFirst = template.getTokenAfter(node.startTag, getTokenOption)
+        const contentLast = template.getTokenBefore(node.endTag, getTokenOption)
+        const type = isMultiline(node, contentFirst, contentLast) ? options.multiline : options.singleline
         if (type === 'ignore') {
           // 'ignore' option
           return
         }
-        const getTokenOption = { includeComments: true, filter: (token) => token.type !== 'HTMLWhitespace' }
-        const contentFirst = template.getTokenAfter(node.startTag, getTokenOption)
-        const contentLast = template.getTokenBefore(node.endTag, getTokenOption)
         const beforeLineBreaks = contentFirst.loc.start.line - node.startTag.loc.end.line
         const afterLineBreaks = node.endTag.loc.start.line - contentLast.loc.end.line
         const expectedLineBreaks = type === 'always' ? 1 : 0
diff --git a/tests/lib/rules/html-content-newline.js b/tests/lib/rules/html-content-newline.js
index 8db90abfa..ee8ccf00d 100644
--- a/tests/lib/rules/html-content-newline.js
+++ b/tests/lib/rules/html-content-newline.js
@@ -40,11 +40,10 @@ tester.run('html-content-newline', rule, {
       </template>`,
     {
       code: `
-        <template>
-          <div class="panel">
-            content
-          </div>
-        </template>`,
+        <template><div class="panel">
+          content
+        </div></template>
+      `,
       options: [{
         singleline: 'always',
         multiline: 'never'
@@ -52,11 +51,10 @@ tester.run('html-content-newline', rule, {
     },
     {
       code: `
-        <template>
-          <div
+        <template><div
             class="panel"
-          >content</div>
-        </template>`,
+          >content</div></template>
+        `,
       options: [{
         singleline: 'always',
         multiline: 'never'
@@ -117,19 +115,17 @@ tester.run('html-content-newline', rule, {
         ignoreNames: ['ignore-tag']
       }]
     },
-    // hasChildElements
-    {
-      code: `
-        <template>
-          <div><div>
+    // multiline contents
+    `
+      <template>
+        <div>
+          <div>
             content
-          </div></div>
-        </template>`,
-      options: [{
-        singleline: 'always',
-        hasChildElements: 'ignore'
-      }]
-    },
+            content
+          </div>
+        </div>
+      </template>
+    `,
     // Ignore if no closing brackets
     `
       <template>
@@ -213,40 +209,36 @@ content
     },
     {
       code: `
-        <template>
-          <div
+        <template><div
             class="panel"
           >
             content
-          </div>
-        </template>
+          </div></template>
       `,
       options: [{
         singleline: 'always',
         multiline: 'never'
       }],
       output: `
-        <template>
-          <div
+        <template><div
             class="panel"
-          >content</div>
-        </template>
+          >content</div></template>
       `,
       errors: [
         {
           message: 'Expected no line breaks after closing bracket of the "div" element, but 1 line break found.',
-          line: 5,
+          line: 4,
           column: 12,
           nodeType: 'HTMLTagClose',
-          endLine: 6,
+          endLine: 5,
           endColumn: 13
         },
         {
           message: 'Expected no line breaks before open bracket of the "div" element, but 1 line break found.',
-          line: 6,
+          line: 5,
           column: 20,
           nodeType: 'HTMLEndTagOpen',
-          endLine: 7,
+          endLine: 6,
           endColumn: 11
         }
       ]
@@ -293,31 +285,24 @@ content
         singleline: 'never'
       }],
       output: `
-        <template><div><!--comment--></div></template>
+        <template>
+          <div><!--comment--></div>
+        </template>
       `,
-      errors: [{
-        message: 'Expected no line breaks after closing bracket of the "template" element, but 1 line break found.',
-        line: 2,
-        column: 19
-      },
-      {
-        message: 'Expected no line breaks after closing bracket of the "div" element, but 1 line break found.',
-        line: 3,
-        column: 16
-
-      },
-      {
-        message: 'Expected no line breaks before open bracket of the "div" element, but 1 line break found.',
-        line: 4,
-        column: 25
+      errors: [
+        {
+          message: 'Expected no line breaks after closing bracket of the "div" element, but 1 line break found.',
+          line: 3,
+          column: 16
 
-      },
-      {
-        message: 'Expected no line breaks before open bracket of the "template" element, but 1 line break found.',
-        line: 5,
-        column: 17
+        },
+        {
+          message: 'Expected no line breaks before open bracket of the "div" element, but 1 line break found.',
+          line: 4,
+          column: 25
 
-      }]
+        }
+      ]
     },
     // one error
     {
@@ -376,7 +361,8 @@ content
           </div></template>
       `,
       options: [{
-        singleline: 'never'
+        singleline: 'never',
+        multiline: 'ignore'
       }],
       output: `
         <template><div>content</div></template>
@@ -392,7 +378,8 @@ content
           content</div></template>
       `,
       options: [{
-        singleline: 'never'
+        singleline: 'never',
+        multiline: 'ignore'
       }],
       output: `
         <template><div>content</div></template>
@@ -403,42 +390,57 @@ content
         column: 24
       }]
     },
-    // hasChildElements
+    // multiline content
     {
       code: `
-        <template><div>content<div>content</div>content</div></template>
+        <template><div>content<div>content
+        content</div>content</div></template>
       `,
       options: [{
-        singleline: 'never',
-        hasChildElements: 'always'
+        singleline: 'never'
       }],
       output: `
         <template>
 <div>
-content<div>content</div>content
+content<div>
+content
+        content
+</div>content
 </div>
 </template>
       `,
-      errors: [{
-        message: 'Expected 1 line break after closing bracket of the "template" element, but no line breaks found.',
-        line: 2,
-        column: 19
-      },
-      {
-        message: 'Expected 1 line break after closing bracket of the "div" element, but no line breaks found.',
-        line: 2,
-        column: 24
-      },
-      {
-        message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
-        line: 2,
-        column: 56
-      },
-      {
-        message: 'Expected 1 line break before open bracket of the "template" element, but no line breaks found.',
-        line: 2,
-        column: 62
-      }]
+      errors: [
+        {
+          message: 'Expected 1 line break after closing bracket of the "template" element, but no line breaks found.',
+          line: 2,
+          column: 19
+        },
+        {
+          message: 'Expected 1 line break after closing bracket of the "div" element, but no line breaks found.',
+          line: 2,
+          column: 24
+        },
+        {
+          message: 'Expected 1 line break after closing bracket of the "div" element, but no line breaks found.',
+          line: 2,
+          column: 36
+        },
+        {
+          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
+          line: 3,
+          column: 16
+        },
+        {
+          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
+          line: 3,
+          column: 29
+        },
+        {
+          message: 'Expected 1 line break before open bracket of the "template" element, but no line breaks found.',
+          line: 3,
+          column: 35
+        }
+      ]
     },
     // empty
     {
@@ -475,7 +477,8 @@ content<div>content</div>content
         </div></template>
       `,
       options: [{
-        singleline: 'never'
+        singleline: 'never',
+        multiline: 'ignore'
       }],
       output: `
         <template><div></div></template>

From c36559d0ea066f454bc3fa03a99ebf4c09b0e4ee Mon Sep 17 00:00:00 2001
From: yosuke ota <otameshiyo23@gmail.com>
Date: Thu, 29 Mar 2018 23:50:16 +0900
Subject: [PATCH 5/6] [update]

---
 README.md                               |  1 +
 docs/rules/html-content-newline.md      | 86 +++++++++++++++++++++++++
 lib/index.js                            |  1 +
 lib/rules/html-content-newline.js       |  2 +-
 tests/lib/rules/html-content-newline.js | 28 ++++----
 5 files changed, 103 insertions(+), 15 deletions(-)
 create mode 100644 docs/rules/html-content-newline.md

diff --git a/README.md b/README.md
index 2467f256e..48a910a17 100644
--- a/README.md
+++ b/README.md
@@ -209,6 +209,7 @@ Enforce all the rules in this category, as well as all higher priority rules, wi
 |:---|:--------|:------------|
 | :wrench: | [vue/html-closing-bracket-newline](./docs/rules/html-closing-bracket-newline.md) | require or disallow a line break before tag's closing brackets |
 | :wrench: | [vue/html-closing-bracket-spacing](./docs/rules/html-closing-bracket-spacing.md) | require or disallow a space before tag's closing brackets |
+| :wrench: | [vue/html-content-newline](./docs/rules/html-content-newline.md) | require or disallow a line break before and after html contents |
 | :wrench: | [vue/prop-name-casing](./docs/rules/prop-name-casing.md) | enforce specific casing for the Prop name in Vue components |
 | :wrench: | [vue/script-indent](./docs/rules/script-indent.md) | enforce consistent indentation in `<script>` |
 
diff --git a/docs/rules/html-content-newline.md b/docs/rules/html-content-newline.md
new file mode 100644
index 000000000..7bb85c317
--- /dev/null
+++ b/docs/rules/html-content-newline.md
@@ -0,0 +1,86 @@
+# require or disallow a line break before and after html contents (vue/html-content-newline)
+
+- :wrench: The `--fix` option on the [command line](http://eslint.org/docs/user-guide/command-line-interface#fix) can automatically fix some of the problems reported by this rule.
+
+## :book: Rule Details
+
+This rule enforces a line break (or no line break) before and after html contents.
+
+
+:-1: Examples of **incorrect** code:
+
+```html
+<div
+  class="panel"
+>content</div>
+```
+
+:+1: Examples of **correct** code:
+
+```html
+<div class="panel">content</div>
+
+<div class="panel">
+  content
+</div>
+
+<div
+  class="panel"
+>
+  content
+</div>
+```
+
+
+## :wrench: Options
+
+```json
+{
+    "vue/html-content-newline": ["error", {
+        "singleline": "ignore",
+        "multiline": "always",
+        "ignoreNames": ["pre", "textarea"]
+    }]
+}
+```
+
+- `singleline` ... the configuration for single-line elements. It's a single-line element if startTag, endTag and contents are single-line.
+    - `"ignore"` ... Don't enforce line breaks style before and after the contents. This is the default.
+    - `"never"` ... disallow line breaks before and after the contents.
+    - `"always"` ... require one line break before and after the contents.
+- `multiline` ... the configuration for multiline elements. It's a multiline element if startTag, endTag or contents are multiline.
+    - `"ignore"` ... Don't enforce line breaks style before and after the contents.
+    - `"never"` ... disallow line breaks before and after the contents.
+    - `"always"` ... require one line break before and after the contents. This is the default.
+- `ignoreNames` ... the configuration for element names to ignore line breaks style.
+    default `["pre", "textarea"]`
+
+
+:-1: Examples of **incorrect** code:
+
+```html
+/*eslint vue/html-content-newline: ["error", { "singleline": "always", "multiline": "never"}] */
+
+<div class="panel">content</div>
+
+<div
+  class="panel"
+>
+  content
+</div>
+```
+
+:+1: Examples of **correct** code:
+
+```html
+/*eslint vue/html-content-newline: ["error", { "singleline": "always", "multiline": "never"}] */
+
+<div class="panel">
+  content
+</div>
+
+<div
+  class="panel"
+>content</div>
+```
+
diff --git a/lib/index.js b/lib/index.js
index d96fd38e0..4cddc194e 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -12,6 +12,7 @@ module.exports = {
     'comment-directive': require('./rules/comment-directive'),
     'html-closing-bracket-newline': require('./rules/html-closing-bracket-newline'),
     'html-closing-bracket-spacing': require('./rules/html-closing-bracket-spacing'),
+    'html-content-newline': require('./rules/html-content-newline'),
     'html-end-tags': require('./rules/html-end-tags'),
     'html-indent': require('./rules/html-indent'),
     'html-quotes': require('./rules/html-quotes'),
diff --git a/lib/rules/html-content-newline.js b/lib/rules/html-content-newline.js
index 9f5908e5b..3483b5c39 100644
--- a/lib/rules/html-content-newline.js
+++ b/lib/rules/html-content-newline.js
@@ -128,7 +128,7 @@ module.exports = {
               start: contentLast.loc.end,
               end: node.endTag.loc.start
             },
-            message: 'Expected {{expected}} before open bracket of the "{{name}}" element, but {{actual}} found.',
+            message: 'Expected {{expected}} before opening bracket of the "{{name}}" element, but {{actual}} found.',
             data: {
               name: node.name,
               expected: getPhrase(expectedLineBreaks),
diff --git a/tests/lib/rules/html-content-newline.js b/tests/lib/rules/html-content-newline.js
index ee8ccf00d..7f56d1463 100644
--- a/tests/lib/rules/html-content-newline.js
+++ b/tests/lib/rules/html-content-newline.js
@@ -162,7 +162,7 @@ content
           endColumn: 12
         },
         {
-          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before opening bracket of the "div" element, but no line breaks found.',
           line: 5,
           column: 19,
           nodeType: 'HTMLEndTagOpen',
@@ -198,7 +198,7 @@ content
           endColumn: 30
         },
         {
-          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before opening bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 37,
           nodeType: 'HTMLEndTagOpen',
@@ -234,7 +234,7 @@ content
           endColumn: 13
         },
         {
-          message: 'Expected no line breaks before open bracket of the "div" element, but 1 line break found.',
+          message: 'Expected no line breaks before opening bracket of the "div" element, but 1 line break found.',
           line: 5,
           column: 20,
           nodeType: 'HTMLEndTagOpen',
@@ -267,7 +267,7 @@ content
           column: 16
         },
         {
-          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before opening bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 30
         }
@@ -297,7 +297,7 @@ content
 
         },
         {
-          message: 'Expected no line breaks before open bracket of the "div" element, but 1 line break found.',
+          message: 'Expected no line breaks before opening bracket of the "div" element, but 1 line break found.',
           line: 4,
           column: 25
 
@@ -349,7 +349,7 @@ content
       `,
       errors: [
         {
-          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before opening bracket of the "div" element, but no line breaks found.',
           line: 4,
           column: 18
         }
@@ -368,7 +368,7 @@ content
         <template><div>content</div></template>
       `,
       errors: [{
-        message: 'Expected no line breaks before open bracket of the "div" element, but 1 line break found.',
+        message: 'Expected no line breaks before opening bracket of the "div" element, but 1 line break found.',
         line: 2,
         column: 31 }]
     },
@@ -426,17 +426,17 @@ content
           column: 36
         },
         {
-          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before opening bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 16
         },
         {
-          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before opening bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 29
         },
         {
-          message: 'Expected 1 line break before open bracket of the "template" element, but no line breaks found.',
+          message: 'Expected 1 line break before opening bracket of the "template" element, but no line breaks found.',
           line: 3,
           column: 35
         }
@@ -465,7 +465,7 @@ content
           column: 16
         },
         {
-          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before opening bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 16
         }
@@ -490,7 +490,7 @@ content
           column: 24
         },
         {
-          message: 'Expected no line breaks before open bracket of the "div" element, but 1 line break found.',
+          message: 'Expected no line breaks before opening bracket of the "div" element, but 1 line break found.',
           line: 2,
           column: 24
 
@@ -525,7 +525,7 @@ content
           column: 16
         },
         {
-          message: 'Expected 1 line break before open bracket of the "div" element, but 2 line breaks found.',
+          message: 'Expected 1 line break before opening bracket of the "div" element, but 2 line breaks found.',
           line: 5,
           column: 20
         }
@@ -555,7 +555,7 @@ content
           column: 16
         },
         {
-          message: 'Expected 1 line break before open bracket of the "div" element, but no line breaks found.',
+          message: 'Expected 1 line break before opening bracket of the "div" element, but no line breaks found.',
           line: 3,
           column: 27
         }

From cdf1349ba7aa0cbdbf51e96a6b15e7911a62cfd7 Mon Sep 17 00:00:00 2001
From: yosuke ota <otameshiyo23@gmail.com>
Date: Thu, 29 Mar 2018 23:53:24 +0900
Subject: [PATCH 6/6] [update] doc

---
 docs/rules/html-content-newline.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/docs/rules/html-content-newline.md b/docs/rules/html-content-newline.md
index 7bb85c317..64e05616e 100644
--- a/docs/rules/html-content-newline.md
+++ b/docs/rules/html-content-newline.md
@@ -52,7 +52,7 @@ This rule enforces a line break (or no line break) before and after html content
     - `"ignore"` ... Don't enforce line breaks style before and after the contents.
     - `"never"` ... disallow line breaks before and after the contents.
     - `"always"` ... require one line break before and after the contents. This is the default.
-- `ignoreNames` ... the configuration for element names to ignore line breaks style.
+- `ignoreNames` ... the configuration for element names to ignore line breaks style.  
     default `["pre", "textarea"]`