0xD34F

Не могу понять как правильно написать цикл перебора.

Цикл не нужен, метод attr умеет принимать функцию и применять её к каждому элементу набора:

$('.hello').attr('data-class', function() {
  return [...this.classList].filter(n => n !== 'hello');
});

И без jquery - цикл в явном виде тоже не нужен:

document.querySelectorAll('.hello').forEach(n => {
  n.dataset.class = n.className.replace(/(^| )hello( | $)/, ' ').trim();
});

Одно непонятно - гарантируется ли, что классов будет всего два? Что если их окажется больше?

Но если точно известно, что классов будет всегда два, и порядок тоже гарантирован, то можно буквально хватать "второй" класс:

$('.hello').attr('data-class', function() {
  return this.classList[1];
});

или

document.querySelectorAll('.hello').forEach(n => {
  n.dataset.class = n.className.split(' ').pop();
});

Array.prototype.push.apply(
  arr,
  newArr.filter(n => !arr.some(m => m.trade_id === n.trade_id))
);

или

const pushDiff = (target, source, key = n => n) =>
  target.push(...source.filter(function(n) {
    return !this.has(key(n));
  }, new Set(target.map(key))));


pushDiff(arr, newArr, n => n.trade_id);

Уберите setTimeout. Из-за него с запозданием отображается сообщение, соответствующее одному из предыдущих результатов поиска, когда ничего найдено не было: запрос - ничего не найдено - setTimeout на показ сообщения об отсутствии результатов - запрос - результаты есть, они отображаются - показ сообщения об отсутствии результатов.

Используйте mouseenter вместо mouseover. Ну и нелишним будет ознакомиться с вот этим.

p:nth-child(n + 2) {

или

p:not(:first-child) {

или

p + p {

Маркерам добавляются идентификаторы - свойство с уникальным значением, а "элементам вне карты" добавляется соответствующий data-атрибут. При клике достаётся значение идентификатора, по значению data-атрибута определяется элемент, которому надо добавить класс. Всё.

Перечитайте внимательно, что вы там у себя понаписали - если значение больше нуля, то пустая строка, иначе 'positive'. Вам вроде бы наоборот надо. То же самое и со вторым условием.

А вообще, можно и без условий:

['negative', '', 'positive'][Math.sign(article.total_comments.length) + 1]

https://github.com/RobinHerbots/Inputmask

.box > p {
  display: inline-block;
}

или

.box {
  display: flex;
}

В main.js:

import Vue from 'vue/dist/vue.esm.js'

В store.js:

import Vue from 'vue'

Вы бы определились, какую сборку vue хотите использовать. А то сейчас подключаете vuex к одной, а реально используете другую. Естественно, $store оказывается undefined.

нет

можно показывать другую картинку с помощью content (например), но работать будет не везде

.row {
  counter-reset: list -1;
}

.row a::before {
  content: counter(list);
  counter-increment: list;
}

.row a:first-child::before,
.row a:last-child::before {
  content: "";
}

https://jsfiddle.net/Ltgybcq3/1/

или

.row {
  counter-reset: list 0;
}

.row a:not(:first-child, :last-child)::before {
  content: counter(list);
  counter-increment: list;
}

https://jsfiddle.net/Ltgybcq3/2/

Чтобы увидеть завершение анимации прогресс бара, удалять его надо по событию transitionend. Добавьте отдельную переменную, которая будет отвечать за отображение прогресс бара, и меняйте её в соответствующем обработчике, как-то так.

const inputSelector = '#kmOutMKAD';
const radioSelector = '[name="delivery"]';
const disabledValue = '1';

$(radioSelector).on('change', function() {
  $(inputSelector).prop('disabled', this.value === disabledValue);
});

// или

const input = document.querySelector(inputSelector);
const radios = document.querySelectorAll(radioSelector);
const onChange = e => input.disabled = e.target.value === disabledValue;
radios.forEach(n => n.addEventListener('change', onChange));

https://jsfiddle.net/t4oc6kd3/

А зачем такие сложности? Пусть у вас будет индекс редактируемого элемента, ну и всё:

$scope.edit = function(index) {
  $scope.active = index;
};

ng-disabled="$index !== active"

overlay.addEventListener('click', function(e) {
  if ([ '#close-menu', 'li > a' ].some(n => e.target.matches(n))) {
    overlay.classList.remove('show-menu');
  }
});

const
  text = 'Sport',
  str = 'port';

console.log(text.replace(new RegExp(`(${str})`, 'g'), '#$1#'));

UPD.

или есть что-то оптимальней ?

Не знаю, что вы подразумеваете под словом оптимальней, но можно просто text.replace(str, `#${str}#`). Правда, в отличие от регулярки - множественную замену так сделать не получится.