Оптимизировать код

[ksa]

Так наверно должно быть

У Ubivectel там разные селекторы...

#scolor, .overlayo, #kide_x_s, #KIDE_mensaje_borrar

[ksa]

Вроде ведь одно и то же, что не так делаю

Нет. Не одно и то же.
У тебя всякий раз разный набор селекторов

#scolor -> #KIDE_opciones, .overlayo
.overlayo -> .KIDE_mostrar, .overlayo
#kide_x_s, #KIDE_mensaje_borrar -> .overlayo

[ksa]

Автор не дает нам шанс проверить...
Предложу такой вариант.

let scolor = document.querySelectorAll('#scolor, .overlayo, #kide_x_s, #KIDE_mensaje_borrar').forEach(el => {
	const a = ['.overlayo']
	if (el.id === 'scolor') a.push('#KIDE_opciones')
	if (el.classList.contains('overlayo')) a.push('.KIDE_mostrar')
	const val = a.join(', ')
	el.onclick = function () {
		document.querySelectorAll(val).forEach(elem => elem.style.display = "none");
	};
})

[Ubivectel]

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

[Ubivectel]

Проверить тут: https://noviy-kovcheg.ru/chat этот код закрывает модальные окна "цвет" и при правом клике на ник в окне сообщений

[Ubivectel]

let scolor = document.querySelectorAll('#scolor, .overlayo, #kide_x_s, #KIDE_mensaje_borrar').forEach(el => {
	const a = ['.overlayo']
	if (el.id === 'scolor') a.push('#KIDE_opciones')
	if (el.classList.contains('overlayo')) a.push('.KIDE_mostrar')
	const val = a.join(', ')
	el.onclick = function () {
		document.querySelectorAll(val).forEach(elem => elem.style.display = "none");
	};
})

Ураааа! Заработало! Всем спасибо товарищи, пошел "курить" как именно все работает.