Вы только назначаете слушателю кнопку «Да», поэтому ваш слушатель будет работать только для вашей кнопки «Да»:
//Listener attached only to yes
yes.addEventListener("click", function(){
if (yes.clicked == true){
box.style.backgroundColor = "red";
} if(no.clicked == true) {
box.style.backgroundColor = "green";
}
});
Так что вам не нужен оператор if для вашей цели, вам нужно только различные слушатели:
box.addEventListener("click", function(){
alert("box clicked!");
});
yes.addEventListener("click", function(){
box.style.backgroundColor = "red";
)};
no.addEventListener("click", function(){
box.style.backgroundColor = "green";
});
И если ваши кнопки находятся внутри коробки, вы можете сделать это вместо верхней функциональности:
box.addEventListener("click", function(ev){
if(ev.currentTarget.id == "yes"){
box.style.backgroundColor = "red";
}else if(ev.currentTarget.id == "no"){
box.style.backgroundColor = "green";
}
});