Вы ошибочно в теле второго условия else-if
обращались к attr.removeAttribute
.
А вторая ошибка в том, что первое условие выполнялось всегда и выполнение не доходило до условия, в котором удалялся атрибут.
list.addEventListener('click', function(e){
let target = e.target;
let currentColor = target.style.color;
if ( target.tagName == 'LI' ) {
target.style.color = ( currentColor === "red" ) ? "black" : "red";
}
/*
if(target.tagName == 'LI' && !attr){
target.style.color = 'red'; //добавили
} else if(attr){
target.removeAttribute('style');
} else {
return false;
}
*/
});
<ul id="list">
<li my-attr="1">1</li>
<li my-attr="2">2</li>
<li my-attr="3">3</li>
<li my-attr="4">4</li>
<li my-attr="5">5</li>
<li my-attr="6">6</li>
<li my-attr="7">7</li>
<li my-attr="8">8</li>
<li my-attr="9">9</li>
<li my-attr="10">10</li>
</ul>