Når du skal sammenligne om to værdier er ens skal du bruge to =. En = bruges
til at angive en variabels værdi.
if(todo="open") => todo får her værdien "open"
if(todo=="open") => der bliver her forespurgt om todo har værdien "open"
--
Marten Ølgaard
http://www.comonto.dk
http://www.beerrater.dk
http://www.b2net.dk
"Peter Nielsen" <pedn@pedn.dk> skrev i en meddelelse
news:46c564c3$0$2714$edfadb0f@dread15.news.tele.dk...
> Hej NG
>
> Jeg er bare en klovn til javascript, så vil høre om i kan finde min fejl
> :)
>
> Jeg har et layer som jeg gerne vil have til at komme frem som en slags
> popupboks på skærmen i ved.. med visibility: hidden
>
> function test(todo) {
> if(todo="close") {
> document.getElementById("layer1").style.visibility = "hidden";
> document.getElementById("layer1").style.position = "absolute";
> document.getElementById("layer1").style.top = "200px";
> document.getElementById("layer1").style.right = "100px";
> }
> }
>
> function test2(todo) {
> if(todo="open") {
> document.getElementById("layer1").style.visibility = "visible";
> document.getElementById("layer1").style.position = "absolute";
> document.getElementById("layer1").style.top = "200px";
> document.getElementById("layer1").style.right = "100px";
> }
> }
>
>
> Når layeret hedder layer1 og man kalder det med javascript:test(close) og
> test2(open)
>
> Så virker lortet...
>
> Men nu vil jeg jo gerne være smart og smide det lidt sammen... og prøve at
> kalde alt med javascript:test(open,layer1) og test(close,layer1) så jeg
> ikk ska ha 2 functions og kan have flere layers...
>
> så ville jeg jo mene at javascripten så sådan her ud:
>
> function test(todo,fieldName) {
> if(todo="close") {
> document.getElementById(fieldName).style.visibility = "hidden";
> document.getElementById(fieldName).style.position = "absolute";
> document.getElementById(fieldName).style.top = "200px";
> document.getElementById(fieldName).style.right = "100px";
> }
>
>
> if(todo="open") {
> document.getElementById(fieldName).style.visibility = "visible";
> document.getElementById(fieldName).style.position = "absolute";
> document.getElementById(fieldName).style.top = "200px";
> document.getElementById(fieldName).style.right = "100px";
> }
> }
>
> Men det virker ikke... nogle der kan hjælpe :)
>
> MVH
>
> Peter Nielsen
>
>