[voraa]

Я вообще не понимаю, как это у вас работает
Возьмем строки 17- 23. Когда первый раз выполняется. Допустим клик по элементу с id "1"

let attr = itemimg.getAttribute('id'); // "1"
arropen.unshift(`${attr}`); // ["1"]
if (arropen[1] != `${attr}`) { arropen[1] === undefined условие выполняется
document.getElementById(`${arropen[1]}`).style.cssText = 'display: block;'; // getElementById("undefined") вернет null и будет ошибка

(Если attr строка, то зачем писать `${attr}`, когда можно просто attr?)

И, кстати, ошибка в css. Строка 46
nbox-shadow: 3px 3px 3px red;