Åbne og lukke

det er allerede implementeret

lav onClick eventene om fra
<a title="Show/hide the answer" onclick="TJK_doToggleDL(0);return false" href="#">Question 1?</a>

<a title="Show/hide the answer" onclick="TJK_ToggleDLclose();TJK_doToggleDL(0);return false" href="#">Question 1?</a>


dvs du kalder TJK_ToggleDLclose() før du åbner den du skal åbne

du _kan_ måske være nødt til at ændre TJK_ToggleDLclose() til ikke at returnere false

Hvis der kunskal være en åben af gangen kunne du prøve at rette til:'

function TJK_doToggleDL(x){
  var zDD=document.getElementById('TJK_DL').getElementsByTagName('dd');
  var zDT=document.getElementById('TJK_DL').getElementsByTagName('dt');

for(var i=0;i<zDT.length;i++){
  if (i==x) {
    zDD[i].className='showDD';
    zDT[i].className='DTminus';
  }else{
    zDD[i].className='hideDD';
    zDT[i].className='DTplus';
  }
}
}

-- men det er dumt at gå ud fra, at du viser og skjuler ved at sætte klassenavne, da det tvinger browseren til at genberegne hele den visuelle præsentation ...

-- arbejd i stedet med at sætte display eller evt. visibility, da det kun berører eet element ...

-- og så kan du med fordel opbevare et tidligere åbent element, så det kan lukkes, når et nyt skal åbnes:

var oldDD = oldDT = null;
function TJK_doToggleDL(x){
  if(oldDD){
    oldDD.style.display = "none";
    oldDT.style.backgroundImage = "img_expand.gif";
  }
  oldDD = document.getElementById('TJK_DL').getElementsByTagName('dd')[x];
  oldDT = document.getElementById('TJK_DL').getElementsByTagName('dt')[x];
  oldDD.style.display = "block";
  oldDT.style.backgroundImage = "img_contract.gif";
}

-- på den måde har du overhovedet kun fat i 4 elementer, som får ændret en enkelt (eller flere, det er der ikke noget galt for !-) style-property, hvor du i det andet script har fat i en hulens masse, og hver gang potentielt sætter browseren igang med en genberegning !-)

jhe-ting, Det skal også være sådan at man kan lukke et spørgsmål, altså at der ikke er nogen som står åben.

roenving, Ja ; -) Jeg forstår nul og niks, og har slet ikke tid til at sætte mig ind i javascript for tiden :-(

OK - tweak - Hvis du kan undvære "Open all," kan det gøres let - ellers vil det være en kompliceret affære. ;)

Det kan jeg, for den funktion skal jeg slet ikke bruge. ;-)

- Men roenving har ret i at det er ineffektiv kode når 'className' rettes på mange elementer ad gangen.
TJK modulet bruger desuden den "foragtede" (dvs. ineffektive og ustandardiserede) innerHTML  !!!
Find evt. en DOM kompatibel pakke.

Hvis jeg forstår dit ønske rigtigt så vil du gerne have en liste hvor der højst kan være et åbent element.
Aldrig mere end ét.

Jeg kigger lige på det... de må kunne laves 'rigtigt' med valid kode.

jhe-ting, du her ret, aldrig mere end et spørgsmål åben, og så skal det være sådan at man kan lukke så der ikke er nogen åben. funktion med åben alle, den har jeg ikke brug for.

Og ja, valid kode vil helt sikkert være et plus, da alt andet er det.

Well - jeg har ikke renset resten af koden, men du kan da prøve dette 'hack' som det jeg viste tidligere:

function TJK_doToggleDL(x){
  var zDD=document.getElementById('TJK_DL').getElementsByTagName('dd');
  var zDT=document.getElementById('TJK_DL').getElementsByTagName('dt');   
  for(var i=0;i<zDT.length;i++){
    if (i==x) {
      zDD[x].className=(zDD[x].className=='hideDD')?'showDD':'hideDD';
      zDT[x].className=(zDT[x].className=='DTplus')?'DTminus':'DTplus';   
    } else {
      zDD[i].className='hideDD';
      zDT[i].className='DTplus';
    }
  }
}

Tak for det :-)

Ahh, har lige et sidespørgsmål, kan man vælge at have spørgsmål 1 åbnet som standard ved sideindlæsningen?

function TJK_ToggleDL(){
if (document.getElementById && document.getElementsByTagName){           
    var zDT=document.getElementById('TJK_DL').getElementsByTagName('dt');
    var zDD=document.getElementById('TJK_DL').getElementsByTagName('dd');
    var ToggleON = document.getElementById('TJK_ToggleON');
    var ToggleOFF = document.getElementById('TJK_ToggleOFF');   
    if (ToggleON && ToggleOFF){// Show All - Hide All "links"
        ToggleON.onclick = TJK_ToggleDLopen;
        ToggleON.title = "Show all answers";
        ToggleON.href = "#";       
        ToggleOFF.onclick = TJK_ToggleDLclose;   
        ToggleOFF.title = "Hide all answers";
        ToggleOFF.href = "#";       
    }
    for(var i=0;i<zDT.length;i++){
        var zContent = zDT[i].innerHTML;
        var zHref = "<a href='#' onclick=\"TJK_doToggleDL("+i+");return false\" title='Show/hide the answer'>";
        zDT[i].innerHTML = zHref + zContent + "</a>";
        zDD[i].className='hideDD';
        zDT[i].className='DTplus';
        }
    }
    zDD[0].className='showDD';
    zDT[0].className='DTminus';
}

Endnu et hurtigt 'hack' ... tak for points :)

Tak :-)

<ole>

Dér røg det valide til gengæld endegyldigt. Nu er det både uhensigtsmæssigt og invald kode. Brug roenvings kode i stedet  :)

/mvh
</bole>

ole, du ved sikkert hvad du snakker om ;-) men det er nu ikke helt ved siden af: http://www.hpregnskab.dk/regnskab.php  - noget andet er at jeg ikke kan gøre mig klog på noget som har med javascript at gøre, men jeg kan altså ikke se der er noget som ikke er hensigtsmæssigt? I er nød til at pensle det helt ud for at jeg forstår bare lidt ;-)