chore: lint

Author: Akryum

.eslintignore

@@ -1,2 +1,4 @@
 node_modules/
 dist/
+docs/
+docs-src/

.eslintrc.js

@@ -1,18 +1,25 @@
 module.exports = {
   root: true,
-  parser: 'babel-eslint',
   parserOptions: {
-    sourceType: 'module'
+    parser: 'babel-eslint',
+    ecmaVersion: 2017,
+    sourceType: 'module',
   },
   // https://github.com/feross/standard/blob/master/RULES.md#javascript-standard-style
-  extends: 'standard',
+  extends: [
+    'standard',
+    'plugin:vue/recommended',
+  ],
   // required to lint *.vue files
+  plugins: [
+    'vue',
+  ],
   env: {
     browser: true,
     jest: true,
   },
   // add your custom rules here
-  'rules': {
+  rules: {
     // allow paren-less arrow functions
     'arrow-parens': 0,
     // allow async-await
@@ -24,5 +31,5 @@ module.exports = {
     // beware of returning assignement
     'no-return-assign': 'off',
     'no-extend-native': 'warn',
-  }
+  },
 }

package.json

@@ -13,6 +13,7 @@
     "dev": "cross-env NODE_ENV=development rollup --config build/rollup.config.es.js --watch",
     "prepublishOnly": "npm run test && npm run build",
     "test": "npm run test:types && npm run test:unit",
+    "lint": "eslint . --ext .js,.vue",
     "test:unit": "vue-cli-service test:unit",
     "test:types": "cd ./tests/types && tsc --noEmit"
   },
@@ -53,6 +54,7 @@
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-promise": "^4.1.1",
     "eslint-plugin-standard": "^5.0.0",
+    "eslint-plugin-vue": "^7.4.0",
     "faker": "^4.1.0",
     "highlight.js": "^9.7.0",
     "jest": "^26.6.3",

src/components/Popover.vue

@@ -14,8 +14,8 @@
     </div>
 
     <div
-      ref="popover"
       :id="popoverId"
+      ref="popover"
       :class="[popoverBaseClass, popoverClass, cssClass]"
       :style="{
         visibility: isOpen ? 'visible' : 'hidden',
@@ -37,9 +37,15 @@
             />
           </div>
 
-          <ResizeObserver v-if="handleResize" @notify="$_handleResize" />
+          <ResizeObserver
+            v-if="handleResize"
+            @notify="$_handleResize"
+          />
         </div>
-        <div ref="arrow" :class="popoverArrowClass"></div>
+        <div
+          ref="arrow"
+          :class="popoverArrowClass"
+        />
       </div>
     </div>
   </div>
@@ -499,10 +505,10 @@ export default {
 
       const events = typeof this.trigger === 'string'
         ? this.trigger
-          .split(' ')
-          .filter(
-            trigger => ['click', 'hover', 'focus'].indexOf(trigger) !== -1
-          )
+            .split(' ')
+            .filter(
+              trigger => ['click', 'hover', 'focus'].indexOf(trigger) !== -1,
+            )
         : []
 
       events.forEach(event => {
@@ -675,10 +681,12 @@ export default {
 
 if (typeof document !== 'undefined' && typeof window !== 'undefined') {
   if (isIOS) {
-    document.addEventListener('touchend', handleGlobalTouchend, supportsPassive ? {
-      passive: true,
-      capture: true,
-    } : true)
+    document.addEventListener('touchend', handleGlobalTouchend, supportsPassive
+      ? {
+          passive: true,
+          capture: true,
+        }
+      : true)
   } else {
     window.addEventListener('click', handleGlobalClick, true)
   }
@@ -695,7 +703,7 @@ function handleGlobalTouchend (event) {
 function handleGlobalClose (event, touch = false) {
   // Delay so that close directive has time to set values
   for (let i = 0; i < openPopovers.length; i++) {
-    let popover = openPopovers[i]
+    const popover = openPopovers[i]
     if (popover.$refs.popover) {
       const contains = popover.$refs.popover.contains(event.target)
       requestAnimationFrame(() => {

src/directives/v-close-popover.js

@@ -2,9 +2,11 @@ import { supportsPassive } from '../utils'
 
 function addListeners (el) {
   el.addEventListener('click', onClick)
-  el.addEventListener('touchstart', onTouchStart, supportsPassive ? {
-    passive: true,
-  } : false)
+  el.addEventListener('touchstart', onTouchStart, supportsPassive
+    ? {
+        passive: true,
+      }
+    : false)
 }
 
 function removeListeners (el) {

src/directives/v-tooltip.spec.js

@@ -1,5 +1,4 @@
 import * as VTooltip from './v-tooltip'
-import { destroyTooltip } from './v-tooltip'
 
 jest.mock('../lib/tooltip')
 

src/index.js

@@ -18,7 +18,7 @@ export function install (Vue, options = {}) {
 
   Vue.directive('tooltip', vtooltip)
   Vue.directive('close-popover', vclosepopover)
-  Vue.component('v-popover', Popover)
+  Vue.component('VPopover', Popover)
 }
 
 export const VTooltip = vtooltip

src/lib/tooltip.js

@@ -630,10 +630,12 @@ if (typeof document !== 'undefined') {
     for (let i = 0; i < openTooltips.length; i++) {
       openTooltips[i]._onDocumentTouch(event)
     }
-  }, supportsPassive ? {
-    passive: true,
-    capture: true,
-  } : true)
+  }, supportsPassive
+    ? {
+        passive: true,
+        capture: true,
+      }
+    : true)
 }
 
 /**

src/utils.js

@@ -70,7 +70,7 @@ export let supportsPassive = false
 if (typeof window !== 'undefined') {
   supportsPassive = false
   try {
-    var opts = Object.defineProperty({}, 'passive', {
+    const opts = Object.defineProperty({}, 'passive', {
       get () {
         supportsPassive = true
       },

tests/unit/.eslintrc.js

@@ -1,5 +1,5 @@
 module.exports = {
   env: {
-    jest: true
-  }
-}
+    jest: true,
+  },
+}