0xD34F

Получить элемент:

const $el = $('[data-value]');
// или
const el = document.querySelector('[data-value]');

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

const str = $el.data('value');
// или
const str = $el.attr('data-value');
// или
const str = el.dataset.value;
// или
const str = el.getAttribute('data-value');
// или
const str = el.attributes['data-value'].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',.

Режем введённую пользователем строку на куски:

const termParts = term.trim().toLowerCase().split(/\s+/);

Проверяем наличие каждой части:

if (termParts.every(n => data[i][0].toLowerCase().includes(n))) {

Сбрасывайте значение validate перед проверкой:

validate = true;
if (/* проверка введённых данных */) {
  validate = false;
}

Или вообще избавьтесь от условного оператора (надо только будет инвертировать результат, чтобы проверка выдавала true, когда всё хорошо):

validate = /* проверка введённых данных */

Отсортировать существующий массив:

arr.sort(({ date: a }, { date: b }) => !a ? 1 : !b ? -1 : a - b);
// или
arr.sort((a, b) => (a.date || Infinity) - (b.date || Infinity));

Собрать отсортированный новый:

const sortedArr = arr
  .map(n => [ n, n.hasOwnProperty('date') ? n.date : Infinity ])
  .sort((a, b) => a[1] - b[1])
  .map(n => n[0]);

Так:

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

  return setInterval(function($items) {
    $items.eq(index).hide();
    index = (index + 1) % $items.length;
    $items.eq(index).show();
  }, delay, $(selector).hide());
}

toggleWithInterval('.container1 > .good', 1000);
toggleWithInterval('.container2 > .good', 300);

??

О каком элементе идёт речь:

const id = '2'; // а нормальный id придумать было - нет, никак?

Как до него добраться:

const el = document.getElementById(id);

// или

const el = document.querySelector(`[id="${id}"]`);

// или (нет, это если бы id нормальный был)

const el = document.querySelector('#' + id);

Вот родительский элемент: const parent = el.parentNode;.

Перемещаем:

parent.prepend(el);

// или

parent.children[0].before(el);

// или

parent.insertAdjacentElement('afterbegin', el);

// или

parent.insertBefore(el, parent.firstElementChild);

1. Вам не кажется странным, что блоков 16, а переменная, отвечающая за угол поворота - всего одна?

2. После каждого поворота проверять углы поворота всех блоков.

В общем, держите. Дальше допиливайте сами.

Самый быстрый и простой способ - заменить var i на let i.

Но есть и другие варианты.

Можно перебирать элементы через forEach вместо обычного цикла, индекс будет всегда правильный:

document.querySelectorAll('li').forEach((n, i) => {
  n.addEventListener('click', e => {
    console.log(i);
  });
});

Или, прицепить индексы к элементам в качестве data-атрибутов:

const onClick = e => console.log(e.target.dataset.index);
const li = document.getElementsByTagName('li');

for (let i = 0; i < li.length; i++) {
  li[i].dataset.index = i;
  li[i].addEventListener('click', onClick);
}

Или, вычислять индекс самостоятельно:

document.querySelector('ul').addEventListener('click', e => {
  if (e.target.tagName === 'LI') {
    const i = [...e.currentTarget.children].indexOf(e.target);
    console.log(i);
  }
});

elem.style.opacity += 0.02;

Тип значения - строка. Поэтому += тут делать нельзя. Можно так:

elem.style.opacity = (+elem.style.opacity + 0.02).toFixed(2);

Вы пытаетесь искать соседей для элементов .q - а они друг другу соседями не приходятся, поскольку у каждого есть отдельный родительский элемент .trigger. Так что замените $(this).siblings() на $(this).parent().siblings().

А вообще, можно сделать гораздо проще:

const $q = $('.trigger .q').click(function() {
  $q.not(this).next().slideUp();
  $(this).next().slideToggle();
});