Aetae

document.activeElement

let arr = [5, 3, 8, 1];
function filterRangeInPlace(innerArr,val1,val2) {
  innerArr = innerArr.filter(el => (val1 <= el && el <= val2));  
}

filterRangeInPlace(arr, 1, 4); // удалены числа вне диапазона 1..4

alert( arr) ; // [3, 1]

Так должно быть понятнее.

Воспользоваться уже написанной, и хорошо оттестированной: mergeWith.
Примерно так:

import { isObject, isUndefined, mergeWith } from 'lodash';

export function merge(...args) {
  return mergeWith(...args, (a, b) => {
    if(
      !(isObject(a) && isObject(b)) // хотя бы один не объект
      && !(isUndefined(a) || isUndefined(b)) // оба заданы
      && a !== b // не равны
    ) return () => null
  });
}

Возможно там потребуются ещё какие-то проверки.

Если закрывали те же криворучки, что писали и первую версию, то может прокатить: "><img src="" onerror="alert(1)">.

Простой ответ:
Берём метод:

метод = function(){
  console.log(this)
};

и(не важно как) пихаем его в какие-то объекты.
Получаем:

хрень.метод() // this - хрень
хрень.кишки.метод() // this - хрень.кишки
метод() // this - window

хрень.метод.call(хрень.кишки) // this - хрень.кишки
метод.call(хрень.кишки) // this - хрень.кишки
метод.call(null) // this - null

элемент.addEventListener('оппа', хрень.метод) //  this -элемент
элемент.addEventListener('оппа', метод) //  this -элемент
элемент.addEventListener('оппа', хрень.кишки.продукт.метод) //  this -элемент

По сути this для фукции - это то, что идёт перед точкой, кроме тех случаев когда он переопределён явно. Всё, никакой эзотерики.)

fn.call(!this ? this : obj) - это чушь скорее всего. Если this - falsy, то ставим его как this при вызове функции fn, иначе ставим obj. Falsy this может быть только при явном указании(или в strict режиме, при вызове функции напрямую("без точки перед ней")), и скорее всего передача пустого this в таком случае просто сломает выполнение fn.

Ну ещё следует упомянуть стрелочную фукнцию:

метод = () => {
  console.log(this)
};

это можно считать за явное указание this, потому что this тут всегда привязан тот, что был при создании стрелочной функции, независимо от того как и откуда её вызывают.

navigator.serviceWorker.ready -> return navigator.serviceWorker.ready :)

Обычное скачивание отследить нельзя. Скачивается файл отдельно от страницы.
Можно, но не желательно: скачивать файл через ajax в память, а потом отдавать повторно как blob. Это будет жрать память пропорционально размеру загружаемого файла и всё равно будет не отслеживаемая задержка пока файл из памяти пишется на диск.
Можно, но чисто теоретически: отслеживать чтение файла на сервере и каким-либо способом(напр. websocket) передавать данные о прогрессе на клиент. Реализацию такого извращения скорее всего придётся пилить самому и она нетривиальна.

Читать полученные параметры из адресной строки и открывать модалку, примерно так:

$(document).ready(function () {
  var searchParams = new URLSearchParams(location.search.slice(1));
  if(searchParams.get('popup') === 'show') {
    $("#myModal-" +  searchParams.get('idpopup')).modal();
  }
});

У Number нет length.

Виталий всё правильно сказал - возвращайте объект и работайте с ним.

А так, если код синхронный, ещё есть дедовский вариант хранить в свойстве самой функции:

dataValidation(val) { 
  if (val == 1) {
    dataValidation.lastError = null;
    return true;
  } else {
    dataValidation.lastError = {
      code: 0xA,
      message: 'is not 1'
    };
    return false;
  } 
}


if (!dataValidation(myInputData)) { 
  swal(dataValidation.lastError); 
} else {
  //дальнейшее выполнение функции
}

Вероятно у вас где-то выше объявлено "use strict"; что запрещает, помимо всего прочего, использование не объявленных переменных.
Решение, соответственно, переменные эти объявлять, используя var, let или const, а не мусорить в глобал.

Перехватить ничего нельзя.
Можно по onsubmit:
a) скачивать файл ajax'ом, и после этого отдавать при наличии.
b) делать предварительно HEAD запрос, и только если статус не 404 вызывать реальный submit.
Вариант a не подходит для больших файлов, т.к. жрёт память. Вариант b делает лишний запрос.

Правильный вариант - переработать логику сервера и формы.

Cross-Origin

Нет.
Утрируя: скобочки после имени функции - это вызов этой функции прям сейчас, вон там где ты это написал.
Обработчиком на клик в данном случае, соответственно, назначается то, что функция возвращает. А clearTimeout у нас не возвращает ничего.
В итоге прямо в момент запуска кода cityRemoveTime будет остановлен, а на каждый клик у нас будет происходить ничего.

Это совсем нетривиально, увы.
Базово можно примерно так:

$(document).click(function(event) {
  var selection = window.getSelection();
  if(
    selection.focusOffset // положение каретки не 0
    && selection.focusNode // есть выбранная нода
    && selection.focusNode.nodeType == 3 // выбрана текстовая нода 
    && selection.focusNode.length !== selection.focusOffset // положение каретки не в конце
  ) selection.focusNode.data = 'OK!';
})

только полбуквы первой и последней в тексте не кликабельны будут.)
Естественно тут не учитываются всякие <br>, <i>, <b> и прочие <span>, которые могут нарубить единый, казалось бы, текст. Тут уж вы сами исхитряйтесь как хотите.)

Очевидно, вот здесь:

// ...
        .then((data) => {
            commentList = new CommentList(commentContainer, data)
        })

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

// ...
        .then((data) => {
            if (data.comments && data.comments.length)
                commentList = new CommentList(commentContainer, data)
        })

Все активные стили:
getComputedStyle()
Все активные правила - совсем иная петрушка.