[JavaScript] - Pytanie początkującej

[Kartofelek]

Nie prawda
Nie działało, bo oba warunki były prawidłowe
Skoro w 1 zmieniałeś na 400 to drugi był prawidłowy. Powinieneś w każdym warunku robić przerwanie funkcji zwracając np false czyli np
if (zmienna.style.margin == "20px")
{
zmienna.style.margin = "400px";
return false;
}

Ale to też nie jest dobrze . Różne przeglądarki różnie mogą zwracać wartość marginesu. Może to być np 400px, a może być 400. Dlatego trzeba to rzutować na wartość liczbową:
parseInt(zmienna.style.margin, 10);