0xD34F

<div id="time"></div>

function setTimer(el, template) {
  return setInterval(el => {
    const d = new Date();
    const timeData = Object.fromEntries([
      [ 'h', 'getHours' ],
      [ 'm', 'getMinutes' ],
      [ 's', 'getSeconds' ],
    ].map(n => [ n[0], `${d[n[1]]()}`.padStart(2, 0) ]));

    el.innerHTML = template.replace(
      /\$(\w)/g,
      (m, g1) => timeData.hasOwnProperty(g1) ? timeData[g1] : m
    );
  }, 1000, typeof el === 'string' ? document.querySelector(el) : el);
}


const intervalId = setTimer('#time', '<strong>Сейчас</strong> $h:$m:$s');

preg_replace('/[^a-zа-я ]/ui', '', $str)

function sortChildren(el, key) {
  el.append(...Array
    .from(el.children, n => [ n, key(n) ])
    .sort((a, b) => a[1] - b[1])
    .map(n => n[0])
  );
}


sortChildren(document.querySelector('.menu'), n => +n.dataset.num);

Потому что селекторы атрибутов и псевдоклассов имеют одинаковую специфичность, а :hover у вас расположен выше.

Дело в том, что getElementsByTagName возвращает динамическую коллекцию элементов, т.е. такую, которая автоматически обновляется при добавлении/удалении элементов. Поэтому сейчас у вас происходит следующее: удалили элемент с индексом 0, и у всех остальных элементов изменился индекс на единицу вниз, 0 вместо 1, 1 вместо 2 и т.д. При этом счётчик цикла вы на единицу увеличили, так что на следующей итерации удаляете элемент с индексом 1 - бывший 2. А тот, который был 1 и стал 0 - его вы пропускаете. А после следующего удаления пропускаете ещё один. Ну и т.д. Каждый второй.

Какие тут есть варианты действий:

• Замените document.getElementsByTagName("p") на document.querySelectorAll('p'), так вы получите статическую коллекцию элементов.
  
• Перебирайте содержимое allP от конца к началу:
  
  

  for (let i = allP.length; i--;) {
    body.removeChild(allP[i]);
  }

  
  
• Удаляйте элемент с нулевым индексом, пока есть что удалять:
  
  

  while (allP.length) {
    allP[0].remove();
  }

  
  
  

Ну и ещё кое-что. Обращаться к body по id - безумие, этот элемент существует в единственном экземпляре и ссылка на него доступна как свойство объекта document. Так что id уберите и замените document.getElementById("body") на document.body.

myPieChart.data.datasets[0].data = [ complete, incomplete ];
myPieChart.update();

как улучшить мой код

Подобный код не улучшают, а переписывают.

function fearNotLetter(str) {
  const missing = Array
    .from(str, n => n.charCodeAt(0))
    .find((n, i, a) => n !== a[0] + i);

  return missing && String.fromCharCode(missing - 1);
}

https://jsfiddle.net/5wtopLka/

getCitiesOptions = () => {
  const country = +this.state.country;

  return Object
    .entries(cities)
    .filter(n => n[1].country === country)
    .map(([ id, city ]) => <option key={id} value={id}>{city.name}</option>);
}

Добавьте collision: 'none' в position.

Тут могут помочь order и ~:

<div class="container">
  <img class="center" src="...">
  <img class="left" src="...">
  <img class="right" src="...">
</div>

.container {
  display: flex;
}

.center {
  order: 2;
}
.left {
  order: 1;
  visibility: hidden;
}
.right {
  order: 3;
  visibility: hidden;
}

.center:hover ~ .left,
.center:hover ~ .right {
  visibility: visible;
}

https://jsfiddle.net/561pc9gr/

Делаем просто:

Array.prototype.unique = function() {
  this.splice(0, this.length, ...new Set(this));
  return this;
};

Делаем сложно:

Array.prototype.unique = function(key = n => n) {
  const getKey = key instanceof Function ? key : n => n[key];
  let numDeleted = 0;

  this.forEach(function(n, i, a) {
    a[i - numDeleted] = n;
    const k = getKey(n);
    numDeleted += this.has(k) || !this.add(k);
  }, new Set);

  this.length -= numDeleted;

  return this;
};


[ 1, 1, 1, 2 ].unique() // [1, 2]
[ { id: 3 }, { id: 1 }, { id: 1 }, { id: 3 } ].unique('id') // [{id: 3}, {id: 1}]
[ 'a', 'b', 'c', 'd', 'ab', 'bc' ].unique(n => n.length) // ['a', 'ab']

То и означает - должна корректно обрабатывать слова без гласных.

Есть косяк. Как тут подсказывают, буква "y" считается гласной не всегда.

Ну и длинно как-то, можно же короче:

function translatePigLatin(str) {
  const x = str.search(/[aioue]/);
  return x > 0
    ? `${str.slice(x)}${str.slice(0, x)}ay`
    : `${str}${x ? '' : 'w'}ay`;
}

function myReplace(str, before, after) {
  if (before[0].toUpperCase() === before[0]) {
    after = after[0].toUpperCase() + after.slice(1);
  }

  return str.replace(before, after);
}

Рендерить элементы с задержкой:

data: () => ({
  items: [
    { show: false, ... },
    { show: false, ... },
    ...
  ],
}),
computed: {
  showItems() {
    return this.items.filter(n => n.show);
  },
},
mounted() {
  this.items.forEach((n, i) => setTimeout(() => n.show = true, i * 1000));
},

<transition-group name="fade">
  <div v-for="n in showItems" :key="n.id">{{ n }}</div>
</transition-group>

:class="{ [`delay-${index}s`]: advantage.notThreeLines }"

<iframe :srcdoc="mailContent"></iframe>