From 7e12d7f8e6c7d7c439593d01173a8b12cbbef64a Mon Sep 17 00:00:00 2001
From: ota <y.ohta.z3@future.co.jp>
Date: Tue, 8 Jan 2019 17:24:07 +0900
Subject: [PATCH 1/4] Update: Add `arrowEmptyLine` option to
 `vue/multiline-html-element-content-newline` rule

Close #683
---
 .../multiline-html-element-content-newline.md | 35 ++++++-
 .../multiline-html-element-content-newline.js | 19 +++-
 .../multiline-html-element-content-newline.js | 96 +++++++++++++++++++
 3 files changed, 146 insertions(+), 4 deletions(-)

diff --git a/docs/rules/multiline-html-element-content-newline.md b/docs/rules/multiline-html-element-content-newline.md
index 9fe8d45af..5517831f3 100644
--- a/docs/rules/multiline-html-element-content-newline.md
+++ b/docs/rules/multiline-html-element-content-newline.md
@@ -78,7 +78,8 @@ This rule enforces a line break before and after the contents of a multiline ele
 {
     "vue/multiline-html-element-content-newline": ["error", {
         "ignoreWhenEmpty": true,
-        "ignores": ["pre", "textarea"]
+        "ignores": ["pre", "textarea"],
+        "arrowEmptyLine": false,
     }]
 }
 ```
@@ -87,6 +88,8 @@ This rule enforces a line break before and after the contents of a multiline ele
     default `true`
 - `ignores` ... the configuration for element names to ignore line breaks style.  
     default `["pre", "textarea"]`
+- `arrowEmptyLine` ... if `true`, allow empty line. If disallow multiple empty lines, use [no-multiple-empty-lines] in combination.  
+    default `false`
 
 ### `"ignores": ["VueComponent", "pre", "textarea"]`
 
@@ -109,6 +112,36 @@ This rule enforces a line break before and after the contents of a multiline ele
 
 </eslint-code-block>
 
+### `"arrowEmptyLine": true`
+
+<eslint-code-block fix :rules="{'vue/multiline-html-element-content-newline': ['error', { arrowEmptyLine: true }]}">
+
+```vue
+<template>
+  <!-- ✓ GOOD -->
+  <div>
+    content
+  </div>
+  <div>
+
+    content
+
+  </div>
+
+  <!-- ✗ BAD -->
+  <div>content
+    content</div>
+</template>
+```
+
+</eslint-code-block>
+
+## :books: Further reading
+
+- [no-multiple-empty-lines]
+
+[no-multiple-empty-lines]: https://eslint.org/docs/rules/no-multiple-empty-lines
+
 ## :mag: Implementation
 
 - [Rule source](https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/multiline-html-element-content-newline.js)
diff --git a/lib/rules/multiline-html-element-content-newline.js b/lib/rules/multiline-html-element-content-newline.js
index b92841b84..dbf246181 100644
--- a/lib/rules/multiline-html-element-content-newline.js
+++ b/lib/rules/multiline-html-element-content-newline.js
@@ -22,7 +22,8 @@ function isMultilineElement (element) {
 function parseOptions (options) {
   return Object.assign({
     ignores: ['pre', 'textarea'],
-    ignoreWhenEmpty: true
+    ignoreWhenEmpty: true,
+    arrowEmptyLine: false
   }, options)
 }
 
@@ -69,6 +70,9 @@ module.exports = {
           items: { type: 'string' },
           uniqueItems: true,
           additionalItems: false
+        },
+        arrowEmptyLine: {
+          type: 'boolean'
         }
       },
       additionalProperties: false
@@ -83,6 +87,7 @@ module.exports = {
     const options = parseOptions(context.options[0])
     const ignores = options.ignores
     const ignoreWhenEmpty = options.ignoreWhenEmpty
+    const arrowEmptyLine = options.arrowEmptyLine
     const template = context.parserServices.getTemplateBodyTokenStore && context.parserServices.getTemplateBodyTokenStore()
     const sourceCode = context.getSourceCode()
 
@@ -94,6 +99,14 @@ module.exports = {
         ignores.includes(casing.kebabCase(node.rawName))
     }
 
+    function isInvalidLineBreaks (lineBreaks) {
+      if (arrowEmptyLine) {
+        return lineBreaks === 0
+      } else {
+        return lineBreaks !== 1
+      }
+    }
+
     return utils.defineTemplateBodyVisitor(context, {
       'VElement' (node) {
         if (inIgnoreElement) {
@@ -127,7 +140,7 @@ module.exports = {
 
         const beforeLineBreaks = contentFirst.loc.start.line - node.startTag.loc.end.line
         const afterLineBreaks = node.endTag.loc.start.line - contentLast.loc.end.line
-        if (beforeLineBreaks !== 1) {
+        if (isInvalidLineBreaks(beforeLineBreaks)) {
           context.report({
             node: template.getLastToken(node.startTag),
             loc: {
@@ -150,7 +163,7 @@ module.exports = {
           return
         }
 
-        if (afterLineBreaks !== 1) {
+        if (isInvalidLineBreaks(afterLineBreaks)) {
           context.report({
             node: template.getFirstToken(node.endTag),
             loc: {
diff --git a/tests/lib/rules/multiline-html-element-content-newline.js b/tests/lib/rules/multiline-html-element-content-newline.js
index b1eaa64c4..b3c2a6327 100644
--- a/tests/lib/rules/multiline-html-element-content-newline.js
+++ b/tests/lib/rules/multiline-html-element-content-newline.js
@@ -72,6 +72,43 @@ tester.run('multiline-html-element-content-newline', rule, {
           class="panel">
         </div>
       </template>`,
+    // arrowEmptyLine
+    {
+      code: `
+        <template>
+          <div
+            class="panel">
+
+          </div>
+        </template>`,
+      options: [{ arrowEmptyLine: true, ignoreWhenEmpty: false }]
+    },
+    {
+      code: `
+        <template>
+          <div
+            class="panel">
+
+            contents
+
+          </div>
+        </template>`,
+      options: [{ arrowEmptyLine: true }]
+    },
+    {
+      code: `
+        <template>
+          <div
+            class="panel">
+
+
+            contents
+
+
+          </div>
+        </template>`,
+      options: [{ arrowEmptyLine: true }]
+    },
     // self closing
     `
       <template>
@@ -439,6 +476,65 @@ content
         'Expected 1 line break before closing tag (`</div>`), but 2 line breaks found.'
       ]
     },
+    // arrowEmptyLine
+    {
+      code: `
+        <template>
+          <div
+            class="panel">
+
+          </div>
+          <div
+            class="panel"></div>
+        </template>`,
+      options: [{ arrowEmptyLine: true, ignoreWhenEmpty: false }],
+      output: `
+        <template>
+          <div
+            class="panel">
+
+          </div>
+          <div
+            class="panel">
+</div>
+        </template>`,
+      errors: [
+        'Expected 1 line break after opening tag (`<div>`), but no line breaks found.'
+      ]
+    },
+    {
+      code: `
+        <template>
+          <div>
+
+            content
+            content
+
+          </div>
+          <div>content
+            content</div>
+        </template>
+      `,
+      options: [{ arrowEmptyLine: true }],
+      output: `
+        <template>
+          <div>
+
+            content
+            content
+
+          </div>
+          <div>
+content
+            content
+</div>
+        </template>
+      `,
+      errors: [
+        'Expected 1 line break after opening tag (`<div>`), but no line breaks found.',
+        'Expected 1 line break before closing tag (`</div>`), but no line breaks found.'
+      ]
+    },
     // mustache
     {
       code: `

From 5c97aefa9fe6b4d14eee1284fbfdd3fb1e0f073b Mon Sep 17 00:00:00 2001
From: ota <y.ohta.z3@future.co.jp>
Date: Tue, 8 Jan 2019 17:31:40 +0900
Subject: [PATCH 2/4] remove extra comma

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

diff --git a/docs/rules/multiline-html-element-content-newline.md b/docs/rules/multiline-html-element-content-newline.md
index 5517831f3..4eff96743 100644
--- a/docs/rules/multiline-html-element-content-newline.md
+++ b/docs/rules/multiline-html-element-content-newline.md
@@ -79,7 +79,7 @@ This rule enforces a line break before and after the contents of a multiline ele
     "vue/multiline-html-element-content-newline": ["error", {
         "ignoreWhenEmpty": true,
         "ignores": ["pre", "textarea"],
-        "arrowEmptyLine": false,
+        "arrowEmptyLine": false
     }]
 }
 ```

From a10a36f1b76cd39c5ebff6fc7f6dcb7ca859ff69 Mon Sep 17 00:00:00 2001
From: yosuke ota <otameshiyo23@gmail.com>
Date: Wed, 9 Jan 2019 01:40:09 +0900
Subject: [PATCH 3/4] fixed typo

---
 .../multiline-html-element-content-newline.md      |  8 ++++----
 .../multiline-html-element-content-newline.js      |  8 ++++----
 .../multiline-html-element-content-newline.js      | 14 +++++++-------
 3 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/docs/rules/multiline-html-element-content-newline.md b/docs/rules/multiline-html-element-content-newline.md
index 4eff96743..9d0f854d5 100644
--- a/docs/rules/multiline-html-element-content-newline.md
+++ b/docs/rules/multiline-html-element-content-newline.md
@@ -79,7 +79,7 @@ This rule enforces a line break before and after the contents of a multiline ele
     "vue/multiline-html-element-content-newline": ["error", {
         "ignoreWhenEmpty": true,
         "ignores": ["pre", "textarea"],
-        "arrowEmptyLine": false
+        "allowEmptyLine": false
     }]
 }
 ```
@@ -88,7 +88,7 @@ This rule enforces a line break before and after the contents of a multiline ele
     default `true`
 - `ignores` ... the configuration for element names to ignore line breaks style.  
     default `["pre", "textarea"]`
-- `arrowEmptyLine` ... if `true`, allow empty line. If disallow multiple empty lines, use [no-multiple-empty-lines] in combination.  
+- `allowEmptyLine` ... if `true`, allow empty line. If disallow multiple empty lines, use [no-multiple-empty-lines] in combination.  
     default `false`
 
 ### `"ignores": ["VueComponent", "pre", "textarea"]`
@@ -112,9 +112,9 @@ This rule enforces a line break before and after the contents of a multiline ele
 
 </eslint-code-block>
 
-### `"arrowEmptyLine": true`
+### `"allowEmptyLine": true`
 
-<eslint-code-block fix :rules="{'vue/multiline-html-element-content-newline': ['error', { arrowEmptyLine: true }]}">
+<eslint-code-block fix :rules="{'vue/multiline-html-element-content-newline': ['error', { allowEmptyLine: true }]}">
 
 ```vue
 <template>
diff --git a/lib/rules/multiline-html-element-content-newline.js b/lib/rules/multiline-html-element-content-newline.js
index dbf246181..7376760f5 100644
--- a/lib/rules/multiline-html-element-content-newline.js
+++ b/lib/rules/multiline-html-element-content-newline.js
@@ -23,7 +23,7 @@ function parseOptions (options) {
   return Object.assign({
     ignores: ['pre', 'textarea'],
     ignoreWhenEmpty: true,
-    arrowEmptyLine: false
+    allowEmptyLine: false
   }, options)
 }
 
@@ -71,7 +71,7 @@ module.exports = {
           uniqueItems: true,
           additionalItems: false
         },
-        arrowEmptyLine: {
+        allowEmptyLine: {
           type: 'boolean'
         }
       },
@@ -87,7 +87,7 @@ module.exports = {
     const options = parseOptions(context.options[0])
     const ignores = options.ignores
     const ignoreWhenEmpty = options.ignoreWhenEmpty
-    const arrowEmptyLine = options.arrowEmptyLine
+    const allowEmptyLine = options.allowEmptyLine
     const template = context.parserServices.getTemplateBodyTokenStore && context.parserServices.getTemplateBodyTokenStore()
     const sourceCode = context.getSourceCode()
 
@@ -100,7 +100,7 @@ module.exports = {
     }
 
     function isInvalidLineBreaks (lineBreaks) {
-      if (arrowEmptyLine) {
+      if (allowEmptyLine) {
         return lineBreaks === 0
       } else {
         return lineBreaks !== 1
diff --git a/tests/lib/rules/multiline-html-element-content-newline.js b/tests/lib/rules/multiline-html-element-content-newline.js
index b3c2a6327..6b4dd28eb 100644
--- a/tests/lib/rules/multiline-html-element-content-newline.js
+++ b/tests/lib/rules/multiline-html-element-content-newline.js
@@ -72,7 +72,7 @@ tester.run('multiline-html-element-content-newline', rule, {
           class="panel">
         </div>
       </template>`,
-    // arrowEmptyLine
+    // allowEmptyLine
     {
       code: `
         <template>
@@ -81,7 +81,7 @@ tester.run('multiline-html-element-content-newline', rule, {
 
           </div>
         </template>`,
-      options: [{ arrowEmptyLine: true, ignoreWhenEmpty: false }]
+      options: [{ allowEmptyLine: true, ignoreWhenEmpty: false }]
     },
     {
       code: `
@@ -93,7 +93,7 @@ tester.run('multiline-html-element-content-newline', rule, {
 
           </div>
         </template>`,
-      options: [{ arrowEmptyLine: true }]
+      options: [{ allowEmptyLine: true }]
     },
     {
       code: `
@@ -107,7 +107,7 @@ tester.run('multiline-html-element-content-newline', rule, {
 
           </div>
         </template>`,
-      options: [{ arrowEmptyLine: true }]
+      options: [{ allowEmptyLine: true }]
     },
     // self closing
     `
@@ -476,7 +476,7 @@ content
         'Expected 1 line break before closing tag (`</div>`), but 2 line breaks found.'
       ]
     },
-    // arrowEmptyLine
+    // allowEmptyLine
     {
       code: `
         <template>
@@ -487,7 +487,7 @@ content
           <div
             class="panel"></div>
         </template>`,
-      options: [{ arrowEmptyLine: true, ignoreWhenEmpty: false }],
+      options: [{ allowEmptyLine: true, ignoreWhenEmpty: false }],
       output: `
         <template>
           <div
@@ -515,7 +515,7 @@ content
             content</div>
         </template>
       `,
-      options: [{ arrowEmptyLine: true }],
+      options: [{ allowEmptyLine: true }],
       output: `
         <template>
           <div>

From d18338676ee133d1beca287af46c224adf445dc8 Mon Sep 17 00:00:00 2001
From: ota <y.ohta.z3@future.co.jp>
Date: Wed, 9 Jan 2019 20:31:46 +0900
Subject: [PATCH 4/4] fixed

---
 .../multiline-html-element-content-newline.md      |  8 ++++----
 .../multiline-html-element-content-newline.js      |  8 ++++----
 .../multiline-html-element-content-newline.js      | 14 +++++++-------
 3 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/docs/rules/multiline-html-element-content-newline.md b/docs/rules/multiline-html-element-content-newline.md
index 9d0f854d5..1004c1ee1 100644
--- a/docs/rules/multiline-html-element-content-newline.md
+++ b/docs/rules/multiline-html-element-content-newline.md
@@ -79,7 +79,7 @@ This rule enforces a line break before and after the contents of a multiline ele
     "vue/multiline-html-element-content-newline": ["error", {
         "ignoreWhenEmpty": true,
         "ignores": ["pre", "textarea"],
-        "allowEmptyLine": false
+        "allowEmptyLines": false
     }]
 }
 ```
@@ -88,7 +88,7 @@ This rule enforces a line break before and after the contents of a multiline ele
     default `true`
 - `ignores` ... the configuration for element names to ignore line breaks style.  
     default `["pre", "textarea"]`
-- `allowEmptyLine` ... if `true`, allow empty line. If disallow multiple empty lines, use [no-multiple-empty-lines] in combination.  
+- `allowEmptyLines` ... if `true`, it allows empty lines around content. If you want to disallow multiple empty lines, use [no-multiple-empty-lines] in combination.  
     default `false`
 
 ### `"ignores": ["VueComponent", "pre", "textarea"]`
@@ -112,9 +112,9 @@ This rule enforces a line break before and after the contents of a multiline ele
 
 </eslint-code-block>
 
-### `"allowEmptyLine": true`
+### `"allowEmptyLines": true`
 
-<eslint-code-block fix :rules="{'vue/multiline-html-element-content-newline': ['error', { allowEmptyLine: true }]}">
+<eslint-code-block fix :rules="{'vue/multiline-html-element-content-newline': ['error', { allowEmptyLines: true }]}">
 
 ```vue
 <template>
diff --git a/lib/rules/multiline-html-element-content-newline.js b/lib/rules/multiline-html-element-content-newline.js
index 7376760f5..8e900b6ec 100644
--- a/lib/rules/multiline-html-element-content-newline.js
+++ b/lib/rules/multiline-html-element-content-newline.js
@@ -23,7 +23,7 @@ function parseOptions (options) {
   return Object.assign({
     ignores: ['pre', 'textarea'],
     ignoreWhenEmpty: true,
-    allowEmptyLine: false
+    allowEmptyLines: false
   }, options)
 }
 
@@ -71,7 +71,7 @@ module.exports = {
           uniqueItems: true,
           additionalItems: false
         },
-        allowEmptyLine: {
+        allowEmptyLines: {
           type: 'boolean'
         }
       },
@@ -87,7 +87,7 @@ module.exports = {
     const options = parseOptions(context.options[0])
     const ignores = options.ignores
     const ignoreWhenEmpty = options.ignoreWhenEmpty
-    const allowEmptyLine = options.allowEmptyLine
+    const allowEmptyLines = options.allowEmptyLines
     const template = context.parserServices.getTemplateBodyTokenStore && context.parserServices.getTemplateBodyTokenStore()
     const sourceCode = context.getSourceCode()
 
@@ -100,7 +100,7 @@ module.exports = {
     }
 
     function isInvalidLineBreaks (lineBreaks) {
-      if (allowEmptyLine) {
+      if (allowEmptyLines) {
         return lineBreaks === 0
       } else {
         return lineBreaks !== 1
diff --git a/tests/lib/rules/multiline-html-element-content-newline.js b/tests/lib/rules/multiline-html-element-content-newline.js
index 6b4dd28eb..8f2f707bd 100644
--- a/tests/lib/rules/multiline-html-element-content-newline.js
+++ b/tests/lib/rules/multiline-html-element-content-newline.js
@@ -72,7 +72,7 @@ tester.run('multiline-html-element-content-newline', rule, {
           class="panel">
         </div>
       </template>`,
-    // allowEmptyLine
+    // allowEmptyLines
     {
       code: `
         <template>
@@ -81,7 +81,7 @@ tester.run('multiline-html-element-content-newline', rule, {
 
           </div>
         </template>`,
-      options: [{ allowEmptyLine: true, ignoreWhenEmpty: false }]
+      options: [{ allowEmptyLines: true, ignoreWhenEmpty: false }]
     },
     {
       code: `
@@ -93,7 +93,7 @@ tester.run('multiline-html-element-content-newline', rule, {
 
           </div>
         </template>`,
-      options: [{ allowEmptyLine: true }]
+      options: [{ allowEmptyLines: true }]
     },
     {
       code: `
@@ -107,7 +107,7 @@ tester.run('multiline-html-element-content-newline', rule, {
 
           </div>
         </template>`,
-      options: [{ allowEmptyLine: true }]
+      options: [{ allowEmptyLines: true }]
     },
     // self closing
     `
@@ -476,7 +476,7 @@ content
         'Expected 1 line break before closing tag (`</div>`), but 2 line breaks found.'
       ]
     },
-    // allowEmptyLine
+    // allowEmptyLines
     {
       code: `
         <template>
@@ -487,7 +487,7 @@ content
           <div
             class="panel"></div>
         </template>`,
-      options: [{ allowEmptyLine: true, ignoreWhenEmpty: false }],
+      options: [{ allowEmptyLines: true, ignoreWhenEmpty: false }],
       output: `
         <template>
           <div
@@ -515,7 +515,7 @@ content
             content</div>
         </template>
       `,
-      options: [{ allowEmptyLine: true }],
+      options: [{ allowEmptyLines: true }],
       output: `
         <template>
           <div>