Посмотри, чему равен text.style.display при первом клике. Пустая строка. Чтобы он был равен 'none', явно укажи его с помощью inline-CSS.
Код HTML:
<div id="text" onclick="closediv()" style="display: none;">
А из глобальных стилей можно вообще убрать.
P.S. Незачем дублировать код в двух функциях. Проще и эффективнее передавать в одну функцию флаг open/close.