Przy pierwszym kliknięciu warunek
if (lang.style.display==="none")
nie będzie spełniony (bo nie masz dla tego elementu ustawionego stylu display na none; masz ustawiony styl w klasie CSS, ale NIE na elemencie), więc wykona się:
else
{
lang.style.display="none";
}
Dopiero przy drugim kliknięciu lang.style.display jest ustawiony na "none", więc zostanie przestawiony na "block".
Możesz po prostu dodać w HTML sobie: style="display: none", albo lepiej - zmienić warunek tak, żeby sprawdzał czy lang.style.display jest w ogóle ustawiony (czy jest pustym stringiem).