buenas...
fijate que ninguna de las dos condiciones if se cumple. es decir, estas leyendo una propiedad donde se asume que esta declarada directamente en el elemento. o sea, el objeto style representaria el atributo style, pero el elemento no tiene declarado ningun atributo style, de modo que display siempre te va devolver una cadena vacia. luego en el condicional estas comparando la cadena vacia vs un string literal el cual siempre te va devolver falso. ¿que puedes hacer? pues en lugar de comprobar si el campo es visible (block), comprueba si la propiedad display tiene valor.
Código:
element.style.display = !element.style.display? 'none': '';
