Por que no funciona mi script? muestra y oculta div

[Tecktonikrazy]

Lo que hace es cambiar el titulo y el contenido de una caja llamada jBox, y si esta visible la oculta, si esta oculta la muestra al llamar a la funcion. Este es el codigo:

Código Javascript:

function jBox(title,content){
 
    var jB = document.getElementById('jBox');
    var JTitle = document.getElementById('jB-title');
    var JContent = document.getElementById('jB-content');
    
    if(jB.style.display == 'none') {jBoxOn()} else
    if(jB.style.display == 'block') {jBoxOff()}
    
    function jBoxOn(){
        jB.style.display = 'block';
        JTitle.innerHTML = title;
        Jcontent.innerHTML = content;
        }
    
    function jBoxOff(){
        jB.style.display = 'none';
        JTitle.innerHTML = title;
        Jcontent.innerHTML = content;
    }
    
}
 

Código HTML:

<div id="jBox">
<a href="javascript:;" onclick="jBox('','')">Cerrar</a>
<span id="jB-title">Titulo</span>
<span id="jB-content">Contenido</span>
</div> 

[zerokilled]

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': '';

[Tecktonikrazy]

Tienes razon, no se me ocurrio definir antes el display, gracias! Por otra parte no entiendo bien lo del codigo, si entiendo a que te refieres pero podrias explicarme porfavor para que sirve "!" y tambien "?" y ":" en !element.style.display? 'none': ''; ???

[zerokilled]

son dos operadores. el primero, logical not o inverso logico (!), invierte un valor booleano. display no es un boolean, pero al realizar una operacion booleana javascript convierte el string a boolean. el segundo, operador ternario (?:) es una simplificacion del condicional if/else. el operador tiene el formato condicion ? expresion1: expresion2;. la expresion1 se evalua si la condicion es cierta y la expresion2 si es falso.
https://developer.mozilla.org/en/Jav...ecedence#Table