Merge pull request #53 from asheshv/ESLINT

cheton · Dec 10, 2019 · 7bf037e · 7bf037e
2 parents 246f01d + 557c834
commit 7bf037e
Show file tree

Hide file tree

Showing 4 changed files with 63 additions and 1 deletion.
diff --git a/package.json b/package.json
@@ -9,7 +9,7 @@
     "build": "babel --out-dir ./lib ./src",
     "build-examples": "cd examples; NODE_ENV=production webpack-cli",
     "pack-examples": "npm run build-examples && zip -r examples.zip examples/{dist,vendor,*.html,*.css}",
-    "test": "tap test/*.js --no-timeout --node-arg=--require --node-arg=@babel/register --node-arg=--require --node-arg=@babel/polyfill",
+    "test": "tap test/*.js --no-timeout --node-arg=--require --node-arg=@babel/register --node-arg=--require --node-arg=@babel/polyfill --reporter classic",
     "dist": "webpack-cli; BUILD_ENV=dist webpack-cli",
     "release": "mkdir -p releases; rm -f releases/*; cp -f dist/infinite-tree.css releases/infinite-tree-${npm_package_version}.css; cp -f dist/infinite-tree.js releases/infinite-tree-${npm_package_version}.js; cp -f dist/infinite-tree.min.js releases/infinite-tree-${npm_package_version}.min.js",
     "lint": "npm run eslint && npm run stylint",

diff --git a/src/clusterize.js b/src/clusterize.js
@@ -27,8 +27,11 @@ class Clusterize extends EventEmitter {
     };
 
     scrollElement = null;
+
     contentElement = null;
+
     rows = [];
+
     cache = {};
 
     scrollEventListener = (() => {
@@ -128,13 +131,15 @@ class Clusterize extends EventEmitter {
         addEventListener(this.scrollElement, 'scroll', this.scrollEventListener);
         addEventListener(window, 'resize', this.resizeEventListener);
     }
+
     destroy(clean) {
         removeEventListener(this.scrollElement, 'scroll', this.scrollEventListener);
         removeEventListener(window, 'resize', this.resizeEventListener);
 
         const rows = clean ? this.generateEmptyRow() : this.rows();
         this.setContent(rows.join(''));
     }
+
     update(rows) {
         this.rows = ensureArray(rows);
 
@@ -150,10 +155,12 @@ class Clusterize extends EventEmitter {
         // Restore scroll position
         this.scrollElement.scrollTop = scrollTop;
     }
+
     clear() {
         this.rows = [];
         this.update();
     }
+
     append(rows) {
         rows = ensureArray(rows);
         if (!rows.length) {
@@ -162,6 +169,7 @@ class Clusterize extends EventEmitter {
         this.rows = this.rows.concat(rows);
         this.changeDOM();
     }
+
     prepend(rows) {
         rows = ensureArray(rows);
         if (!rows.length) {
@@ -170,6 +178,7 @@ class Clusterize extends EventEmitter {
         this.rows = rows.concat(this.rows);
         this.changeDOM();
     }
+
     computeHeight() {
         if (!this.rows.length) {
             return {
@@ -203,13 +212,15 @@ class Clusterize extends EventEmitter {
             };
         }
     }
+
     getCurrentClusterIndex() {
         const { blockHeight, clusterHeight } = this.state;
         if (!blockHeight || !clusterHeight) {
             return 0;
         }
         return Math.floor(this.scrollElement.scrollTop / (clusterHeight - blockHeight)) || 0;
     }
+
     generateEmptyRow() {
         const { tag, emptyText, emptyClass } = this.options;
 
@@ -231,6 +242,7 @@ class Clusterize extends EventEmitter {
 
         return [emptyRow.outerHTML];
     }
+
     renderExtraTag(className, height) {
         const tag = document.createElement(this.options.tag);
         const prefix = 'infinite-tree-';
@@ -246,6 +258,7 @@ class Clusterize extends EventEmitter {
 
         return tag.outerHTML;
     }
+
     changeDOM() {
         if (!this.state.clusterHeight && this.rows.length > 0) {
             if (ie && ie <= 9 && !this.options.tag) {
@@ -312,6 +325,7 @@ class Clusterize extends EventEmitter {
             this.contentElement.lastChild.style.height = bottomOffset + 'px';
         }
     }
+
     setContent(content) {
         // For IE 9 and older versions
         if (ie && ie <= 9 && this.options.tag === 'tr') {
@@ -332,6 +346,7 @@ class Clusterize extends EventEmitter {
             this.contentElement.innerHTML = content;
         }
     }
+
     getChildNodes(tag) {
         const childNodes = tag.children;
         const nodes = [];
@@ -343,6 +358,7 @@ class Clusterize extends EventEmitter {
 
         return nodes;
     }
+
     checkChanges(type, value) {
         const changed = (value !== this.cache[type]);
         this.cache[type] = value;