Почему выдает ошибку Uncaught TypeError: Cannot read properties of undefined (reading 'split')?

Question

[Вадимка)))]

Делал расширение, написал код:

let changeColor = document.getElementById('buttonBg');

if (changeColor != null) {
	changeColor.onclick = function(el) {  //ругается на эту строку.
		let colorBg = '#333';
		let colorText = '#fff'

		chrome.tabs.query({active: true, currentWindow: true}, function(tabs) {
			chrome.tabs.executeScript(
				tabs[0].id,
				{code: 'document.body.style.backgroundColor = "' + colorBg + '";document.body.style.color = "' + colorText + '"'}
			);
		})
	}
}

Подключаю код не в html-странице, а в manifest.json, поэтому onload и defer использовать не могу.

Answer 1

[Aetae]

onload - можешь. Только не onload addEventListener('load', ...).
Правда это и не нужно, для content_scripts указывается когда его запускать в параметре run_at.

Точно также не onclick, а addEventListener('click', ...). Вообще забудь об on<event> когда лезешь на чужие страницы. Такой обработчик может быть только один, в то время как addEventListener сколько угодно.

Ну и я нигде не вижу в твоём коде split, на который жалуется ошибка, так что где-то ты привираешь.

Comments

[Вадимка)))]

Насчет "split" я тоже удивился, chrome почему-то на все мои коды одной этой ошибкой ругается.

[Aetae]

Вадимка))), сделай текстовый поиск по файлам, может ты где-то его случайно в коде оставил?)

[Вадимка)))]

Скорее всего, ошибка хрома, я в предыдущем расширении "split" использовал.

[Вадимка)))]

Потому-что вчера весь день оно выдавало эту ошибку, сегодня не выдает

[Aetae]

Вадимка))), хоть ошибка хрома и возможна, но к таким вариантам объяснения стоит переходить только десять раз перепроверив свой код.)

[Вадимка)))]

А как ты еще объяснишь то, что код не работал, а после перезагрузки компа он заработал сразу же?

[Aetae]

Вадимка))), ну может быть ты сделал что-то не так? Например оставил старую версию висеть подключая новую, хз.
Перезагрузка всё спишет.)

[Вадимка)))]

Чел, хватит на меня наезжать. Это уже не ответ на вопрос, а просто троллинг.

[Aetae]

Вадимка))), блин, ну серьёзно. Ты уводишь тему в сторону что "все вокруг виноваты", а я пытаюсь вернуть твои мысли в правильную для программиста сторону, что мб и выглядит как наезд.
По сути:
1. Если есть ошибка - у неё есть причина.
2. Причина любой ошибки - программист.
3. Есть шанс что причина ошибки НЕ программист, но он пренебрежимо мал и стоит о нём забыть, дабы не тешить себя иллюзиями.
4. Перезагрузка без выяснения причины проблемы - НЕ решает проблему, а откладывает её до следующего раза.