set, unset var

det ville nok hjælpe at kigge lidt på den første linie i thread_expand...
Du sætter en variabel til (altid) true, hvorefter du bruger den variabel til at afgøre om du skal gøre det ene eller det andet... Spm er 1½ time gammelt nu, så jeg håber du allerede har fundet ud af dét..??

hehe... nej, har stadig ikke fattet det.. LOL

du må nok hellere lige fortælle hvad jeg gør galt ;)

Okay. Første linie i din funktion siger:
expand[thread] = true;

Derefter kommer:
if(expand[thread]) {
// fold ud
...
} else {
// fold ind
...
}

Men ligegyldig hvad, så kommer funktionen *aldrig* til "fold ind", for i første linie står der jo, at expand[thread] *altid* skal være "true".

Hvad sker der hvis du erstatter første linie med

if ('undefined' == typeof expand[thread]) expand[thread] = true;

hehe... du har jo styr på sagerne *g*

gider du ikke lige forklare mig hvad den linje gør? :)

samtidig kan du jo lige lave et svar... hehe

'typeof' er en operator, som returnerer en streng, afhængig af indholdet af en variabel. Hvis variablen ikke er blevet defineret, er den 'undefined'. Andre mulige værdier er 'function', 'number' og 'string'.

M.a.o. expand[thread] bliver sat TRUE, kun hvis det er første gang der bliver pillet ved den.

tark ;)

kan du ikke også lige give en lille beskrivelse af:
expand[thread] = !expand[thread];

vil gerne vide hvad det er som jeg laver og ikke bare skrive noget fordi det virker

btw:

function thread_expand(thread)
{
    var img = getElementById('img'+thread);
    var div = getElementById('div'+thread);
    var td = getElementById('td'+thread);
   
    if(div.style.display == 'block')
    {
    // fold ind
        img.src = 'gfx/debate_thread_close.gif';
        div.style.display = 'none';
        td.style.backgroundImage = '';
    }
    else
    {
    // fold ud
        img.src = 'gfx/debate_thread_open.gif';
        div.style.display = 'block';
        td.style.backgroundImage = "url('gfx/debate_thread_bg.gif')";
    }
}

Så drop 'expand' variablen, og test i stedet style.display...
Bemærk at jeg har byttet om på rækkefølgen af fold-u og fold-ind. det er fordi man ikke kan være sikker på hvilken værdi style.display har fra starten (hvis man bruger et stylesheet) derfor er det fedest hvis den ting der skal ske *første gang* står i 'else'.

det virker ikke?!

det forrige:
!noget betyder på javascript'sk NOT noget, dvs det boolsk omvendte:
!true == false && !false == true
! operatoren konverterer også en værdi til boolean, hvis den er noget andet. Så i teorien burde det oprindelige script måske virke, hvis bare man *fjernede* den første linie (expand[thread] = true), og så ændrede den flg if til if (!expand[thread])... altså lod den sidste linie konvertere {'undefined'} til {false}, og derpå {false} til {true} (NOT-operationen).
Hvilket komplet vrøvl! Nørderne længe leve!

nej jeg har heller ikke testet det, bare brug den gamle ver.

dvs. at den er det samme som:

if(expand[thread] == true) expand[thread] == false;
else expand[thread] == true;

det er lidt tricky det sidste der... jeg troede jeg kunne huske hvad det var, jeg har nemlig lavet en lignenede ting en gang.

sådan:

function toggle (id) {
    var e = document.getElementById ('f'+id).style;
    var f = document.images['h'+id];
    if (e.display == 'none') {
        f.src='billeder/gra/foldi.gif';
        e.display = 'block';
    }
    else {
        f.src='billeder/gra/foldu.gif';
        e.display = 'none';
    }
}

men så var der også et <div style="display:none">, for så kan man godt aflæse værdien fra javascript.