Enforce a consistent ES6 coding style

Replace the old (and broken) jshint + jscsrc by eslint and configure
it to support some of the ES6 features.

The command `eslint` currently has one error which is a bug that was
discovered by its static analysis and should be fixed (usage of a
dead object).

2 files changed, 17 insertions, 19 deletions

diff --git a/client/components/mixins/infiniteScrolling.js b/client/components/mixins/infiniteScrolling.js
index df3b8901..0afb33f9 100644
--- a/client/components/mixins/infiniteScrolling.js
+++ b/client/components/mixins/infiniteScrolling.js
@@ -1,37 +1,37 @@
-var peakAnticipation = 200;
+const peakAnticipation = 200;
 
 Mixins.InfiniteScrolling = BlazeComponent.extendComponent({
-  onCreated: function() {
+  onCreated() {
     this._nextPeak = Infinity;
   },
 
-  setNextPeak: function(v) {
+  setNextPeak(v) {
     this._nextPeak = v;
   },
 
-  getNextPeak: function() {
+  getNextPeak() {
     return this._nextPeak;
   },
 
-  resetNextPeak: function() {
+  resetNextPeak() {
     this._nextPeak = Infinity;
   },
 
   // To be overwritten by consumers of this mixin
-  reachNextPeak: function() {
+  reachNextPeak() {
 
   },
 
-  events: function() {
+  events() {
     return [{
-      scroll: function(evt) {
-        var domElement = evt.currentTarget;
-        var altitude = domElement.scrollTop + domElement.offsetHeight;
+      scroll(evt) {
+        const domElement = evt.currentTarget;
+        let altitude = domElement.scrollTop + domElement.offsetHeight;
         altitude += peakAnticipation;
         if (altitude >= this.callFirstWith(null, 'getNextPeak')) {
           this.callFirstWith(null, 'reachNextPeak');
         }
-      }
+      },
     }];
-  }
+  },
 });
diff --git a/client/components/mixins/perfectScrollbar.js b/client/components/mixins/perfectScrollbar.js
index 510a4563..f652f043 100644
--- a/client/components/mixins/perfectScrollbar.js
+++ b/client/components/mixins/perfectScrollbar.js
@@ -1,14 +1,12 @@
 Mixins.PerfectScrollbar = BlazeComponent.extendComponent({
-  onRendered: function() {
-    var component = this.mixinParent();
-    var domElement = component.find('.js-perfect-scrollbar');
+  onRendered() {
+    const component = this.mixinParent();
+    const domElement = component.find('.js-perfect-scrollbar');
     Ps.initialize(domElement);
 
     // XXX We should create an event map to be consistent with other components
     // but since BlazeComponent doesn't merge Mixins events transparently I
     // prefered to use a jQuery event (which is what an event map ends up doing)
-    component.$(domElement).on('mouseenter', function() {
-      Ps.update(domElement);
-    });
-  }
+    component.$(domElement).on('mouseenter', () => Ps.update(domElement));
+  },
 });