Założenie jest takie: klikam na tekst, zmienia się na czerwony. Jeśli jest już czerwony i kliknę znowu, to robi się czarny.
Funkcja jednak wykonuje się w całości, w efekcie tekst zmienia się na czerwony na ułamek sekundy, a później znowu jest czarny. Jak zrobić, żeby działało dobrze? Czego potrzebuję, żeby funkcja przestała się wykonywać po wykonaniu jednej z instrukcji warunkowych?

[HTML] pobierz, plaintext 
function minus(show) { 
	if (document.getElementById(show).style.color = "black") {
		document.getElementById(show).style.color = "red"; 
	}
 
	if (document.getElementById(show).style.color = "red") {
	       document.getElementById(show).style.color = "black"; 
	}
}
[HTML] pobierz, plaintext 

if else

[HTML] pobierz, plaintext 
function minus(show) {
    if (document.getElementById(show).style.color = "black") {
    document.getElementById(show).style.color = "red";
    }
 
    else {
    document.getElementById(show).style.color = "black";
    }
    }
[HTML] pobierz, plaintext 

Teraz tekst robi się czerwony, ale po ponownym kliknięciu nie staje się czarny.

Obydwa kody są poprawne ale mylisz operator przypisania =, z operatorem porównania ==.
W warunkach musisz oczywiście użyć tego drugiego.

Działa, dzięki.
Temat do zamknięcia.