0xD34F

Переписал код с ES5 на ES6

Брехня. Первое что видим, открыв оба варианта - const и шаблонную строку. То есть, у вас везде используются возможности ES6. Во втором варианте (это который якобы ES6) для сохранения контекста зачем-то применяется bind вместо arrow functions.

Теперь какие косяки были вами допущены:

• забыли подключить jquery
  
• в методе clickarea неправильно привязали контекст, в результате пытаетесь вызывать несуществующий метод
  
• в методе _setupDrag неправильно вешаете обработчик mousemove или в методе _finishDrag неправильно снимаете - в первом у вас addEventListener, во втором off
  
• в методе _updateTrackFill неправильно вычисляется новая позиция - должно быть this._length вместо length
  

Вот исправленный вариант.

$('.menu_item').click(function() {
  $('ul', this).slideToggle('slow');
});

Что значит "перегружен"? Как по мне - так нормально.

А вот то, что выдаёт результаты неправильные - это косяк. Четвёртый пример, максимальные элементы - 10 и 24, произведение 240, а у вас 45. Сортировка по умолчанию сравнивает элементы как строки, поэтому максимальными у вас оказываются 5 и 9. Ещё с пустым массивом всё плохо будет.

arr = arr.filter(n => n.id !== ID);

Или, если не хотите создавать новый массив:

arr.splice(arr.findIndex(n => n.id === ID), 1);

Это, конечно, если вы уверены, что элемент с таким id действительно есть. В противном случае перед удалением надо проверить, чем является индекс:

const index = arr.findIndex(n => n.id === ID);
if (index !== -1) {
  arr.splice(index, 1);
}

function toggleClass(selector, className, delay) {
  let index = -1;

  return setInterval($items => {
    $items.eq(index).removeClass(className);
    index = (index + 1) % $items.length;
    $items.eq(index).addClass(className);
  }, delay, $(selector));
}


const intervalId = toggleClass('.wrap div', 'active', 300);

или

function toggleClass(selector, className, delay) {
  const items = document.querySelectorAll(selector);
  let index = ~-items.length;

  return items.length
    ? setInterval(() => {
        items[index].classList.remove(className);
        index = -~index % items.length;
        items[index].classList.add(className);
      }, delay)
    : null;
}

const msgArr = msg.trim().replace(/[-,]/g, '').split(/\s+/);

document.querySelectorAll('.item-parent').forEach(n => {
  n.insertAdjacentHTML('beforeend', '<div class="item-decor"></div>');
});

или

for (const n of document.getElementsByClassName('item-parent')) {
  const div = document.createElement('div');
  div.classList.add('item-decor');
  n.appendChild(div);
}

function prev(el, selector) {
  if (el instanceof HTMLElement) {
    el = el.previousElementSibling;
    return el && (!selector || el.matches(selector)) ? el : null;
  }

  if (el && Number.isInteger(el.length) && el.length >= 0) {
    return Array.prototype.reduce.call(
      el,
      (acc, n) => ((n = prev(n, selector)) && acc.push(n), acc),
      []
    );
  }

  return null;
}


// можно передавать как одиночный элемент (результатом будет тоже элемент или null)...
prev(document.body)

// ... так и коллекции (результатом будет массив)
prev(document.images)
prev(document.querySelectorAll('div'), 'span')

А вообще - загляните-ка сюда.

const $sorted = $('table tr').sort(el => $(el).hasClass('important') ? -1 : 1);
$sorted.appendTo($sorted.parent());

или

const $important = $('table tr.important');
$important.parent().prepend($important);

О каком data-атрибуте идёт речь:

const key = 'value';
const attr = `data-${key}`;
const selector = `[${attr}]`;

Получаем элемент:

const $el = $(selector);
// или
const el = document.querySelector(selector);

Достаём из элемента значение атрибута:

const str = $el.data(key);
// или
const str = $el.attr(attr);
// или
const str = el.dataset[key];
// или
const str = el.getAttribute(attr);
// или
const str = el.attributes[attr].value;

Превращаем полученную строку в массив:

const arr = str.split(',');
// или
const arr = str.match(/\d+/g) || [];
// или
const arr = str.split(/\D/);
// или
const arr = JSON.parse(`[${str}]`); // вместо строк тут будут числа
// или
const arr = eval('[' + str + ']'); // и тут тоже числа

Про всплытие событий слышали?

Обработчик клика на .modal-close не нужен - его убираем; добавляем проверку, кем является целевой элемент:

$('.modal-window').click(function(e) {
  if (e.target.matches('.modal-window, .modal-close')) {
    $(this).hide();
  }
});

Куда, что и в каком количестве надо добавить:

const container = document.querySelector('.dots-wrapper');
const tag = 'div';
const className = 'dot';
const count = 10;

Можно пойти двумя путями:

• Первый - собирание разметки. Для одного элемента она будет выглядеть так:
  
  

  const elementHTML = `<${tag} class="${className}"></${tag}>`;

  
  Чтобы её размножить, есть варианты:
  
  

  const elementsHTML = Array(count).fill(elementHTML).join('');
  // или
  const elementsHTML = Array(count + 1).join(elementHTML);
  // или
  const elementsHTML = elementHTML.repeat(count);

  
  Добавляем:
  
  

  container.innerHTML = elementsHTML;
  // или
  container.insertAdjacentHTML('beforeend', elementsHTML);

  
  
• Второй - создание элементов напрямую. Тут тоже есть разные варианты:
  
  

  for (let i = 0; i < count; i++) {
    const element = document.createElement(tag);
    element.classList.value = className;
    container.insertAdjacentElement('beforeend', element);
  }

  
  или
  
  

  while (container.children.length < count) {
    const element = document.createElement(tag);
    element.classList.add(className);
    container.appendChild(element);
  }

  
  или
  
  

  const element = document.createElement(tag);
  element.className = className;
  container.append(...Array.from({ length: count }, () => element.cloneNode()));

  
  

Ищем готовое или реализовываем самостоятельно:

function throttle(f, delay) {
  let lastCall = -Infinity;

  return function() {
    const now = +new Date;
    if (now - lastCall > delay) {
      lastCall = now;
      return f.apply(this, arguments);
    }
  };
}

Так, что ли?

Можно использовать bind:

setTimeout(function() {
  console.log(this.resorts.join(', '));
}.bind(this), delay);

Или в переменную сохранить текущий контекст до вызова setTimeout и использовать её вместо this:

var that = this;
setTimeout(function() {
  console.log(that.resorts.join(', '));
}, delay);

<button data-block="block-one">69</button>
<button data-block="block-two">187</button>
<button data-block="block-three">666</button>

<div class="block block-one">hello, world!!</div>
<div class="block block-two">fuck the world</div>
<div class="block block-three">fuck everything</div>

$('body').on('click', 'button', function() {
  const $block = $(`.${this.dataset.block}`).toggleClass('show');
  $('.block').not($block).removeClass('show');
});

// или

document.addEventListener('click', ({ target: { dataset: { block } } }) => {
  if (block) {
    document.querySelectorAll('.block').forEach(({ classList: cl }) => {
      cl[cl.contains(block) ? 'toggle' : 'remove']('show');
    });
  }
});

или

<button class="button">69</button>
<button class="button">187</button>
<button class="button">666</button>

<div class="block block-one">hello, world!!</div>
<div class="block block-two">fuck the world</div>
<div class="block block-three">fuck everything</div>

const $blocks = $('.block');
const $buttons = $('.button').click(function() {
  const $block = $blocks.eq($buttons.index(this)).toggleClass('show');
  $blocks.not($block).removeClass('show');
});

// или

const blocks = document.querySelectorAll('.block');
const buttons = document.querySelectorAll('.button');

buttons.forEach(n => n.addEventListener('click', onClick));

function onClick() {
  const index = Array.prototype.indexOf.call(buttons, this);
  blocks.forEach((n, i) => n.classList[i === index ? 'toggle' : 'remove']('show'));
}

html += '<li onclick="location.href=\'' + product.url + '\'">';

или

html += `<li onclick="location.href='${product.url}'">`;

Замените обычную функцию на стрелочную:

fs.open(fname, 'a', (err, file_handle) => {
  if (!err)
    this.file_handle = file_handle;

Или bind используйте. Или можете до вызова fs.open сохранить текущий контекст в переменную, и использовать её внутри коллбека вместо this.

Вместо

$('#given1').on('change', 'select', function(){
  addNewSelect();
});

должно быть

$('#given1').on('change', 'select', addNewSelect);

Ну и второй обработчик аналогично. Кстати, их можно объединить - допускается указывать несколько имён событий, разделив их пробелом. Типа так: .on('change blur',.