Fixes after review

Fix pipelines tests

Fix rubocop

1 files changed, 3 insertions, 11 deletions

diff --git a/app/assets/javascripts/extensions/element.js.es6 b/app/assets/javascripts/extensions/element.js.es6
index 0abe8644f30..328cc57dca0 100644
--- a/app/assets/javascripts/extensions/element.js.es6
+++ b/app/assets/javascripts/extensions/element.js.es6
@@ -1,19 +1,11 @@
 /* global Element */
-/* eslint-disable consistent-return, max-len */
+/* eslint-disable consistent-return, max-len, no-empty, no-plusplus, func-names */
 
 Element.prototype.closest = Element.prototype.closest || function closest(selector, selectedElement = this) {
   if (!selectedElement) return;
   return selectedElement.matches(selector) ? selectedElement : Element.prototype.closest(selector, selectedElement.parentElement);
 };
 
-/* eslint-disable */
-/**
- * .matches polyfill from mdn
- * https://developer.mozilla.org/en-US/docs/Web/API/Element/matches
- *
- * .matches is used in our code.
- * In order to run the tests in Phantomjs we need this polyfill
- */
 if (!Element.prototype.matches) {
   Element.prototype.matches =
       Element.prototype.matchesSelector ||
@@ -22,8 +14,8 @@ if (!Element.prototype.matches) {
       Element.prototype.oMatchesSelector ||
       Element.prototype.webkitMatchesSelector ||
       function (s) {
-        var matches = (this.document || this.ownerDocument).querySelectorAll(s),
-          i = matches.length;
+        const matches = (this.document || this.ownerDocument).querySelectorAll(s);
+        let i = matches.length;
         while (--i >= 0 && matches.item(i) !== this) {}
         return i > -1;
       };