Skift background-color i TD med checkbox

En smule ændring i denne her ...

<script language="javascript" type="text/javascript">
function makeGreen(t){
  var tds = document.getElementsByTagName("td");
  for (i=0;i<tds.length;i++) {
    if (tds[i].id.indexOf("farve")!=-1) {
      tds[i].style.backgroundColor="#FFFFFF";
    }
  }
  t.style.backgroundColor="#00FF00"
}

Har dine td'er som skal skifte farve en særlighed?

-- i den ovenstående kodestump hedder de alle noget med farve, er alle dine som skal skifte farve de eneste, som hedder td_body_?-)

<script language="javascript" type="text/javascript">
function tdShift() {
  var tds = document.getElementsByTagName("td");
  for (i=0;i<tds.length;i++) {
    if (tds[i].name.indexOf("td_body_")!=-1) {
      tds[i].className='checkbox-click';
    }
  }
}
</script>

og med IE4 også

function tdShift() {
  if (document.getElementById) {
    var tds = document.getElementsByTagName('TD');
    for (i=0; i<tds.length; i++) {
      if (tds[i].id.indexOf("td_body_")!=-1) {
        tds[i].className = "checkbox-click";
      }
    }
  }
  if (document.all) {
    var tds = document.all.tags('TD');
    for (i=0; i<tds.length; i++) {
      if (tds[i].name.indexOf("td_body_")!=-1) {
        tds[i].className = "checkbox-click";
      }
    }
  }
}

Dog er det ikke optimalt med klasseskift, det er ret langsomt.

>>pelkjaer

-- det har du vel nok ret i, men brugeren opdager det næppe, hvis hans maskine kører med mere end 200 MHz :O)

-- til gengæld er metoden vel egentlig den anbefalede idag ...

Skulle man så ikke også lige sætte en else ind, så den ikke gør det to gange (IE5+) ...

Jeg ved ikke hvordan jeg giver dig(roenvig) de der points :-(
PS: nu ændrer den sig når man sætter en vinge, men scriptet går ikke tilbage til den "gamle" classe når man fjerner vingen igen.

Det fanme dårlig brugervenlighed. Jeg kan ikke give point :-(

Så skal vi have en skift-parameter at reagere på (også med udgangspunkt i pelkjaers kommentar):

var shiftTD=true;
function tdShift() {
  if (document.getElementById) {
    var tds = document.getElementsByTagName('TD');
    for (i=0; i<tds.length; i++) {
      if (tds[i].id.indexOf("td_body_")!=-1) {
        if (shiftTD){
          tds[i].className = "checkbox-click";
        }else{
          tds[i].className = "td_body";
        }
      }
    }
  }else{
    if (document.all) {
      var tds = document.all.tags('TD');
      for (i=0; i<tds.length; i++) {
        if (tds[i].name.indexOf("td_body_")!=-1) {
          if (shiftTD){
            tds[i].className = "checkbox-click";
          }else{
            tds[i].className = "td_body";
          }
        }
      }
    }
  }
}

hth
jes

Det er fordi han ikke har svaret. Han svarer sjældent.

og jeg glemte:

var shiftTD=true;
function tdShift() {
  shiftTD=!shiftTD;

Jeg kan ikke få det nye du har sendt til at virke, roenvig :-(

Hvis vi tager min oprindelige:

<script language="javascript" type="text/javascript">
var shiftTD=true;
function tdShift() {
  var tds = document.getElementsByTagName("td");
  for (i=0;i<tds.length;i++) {
    if (tds[i].name.indexOf("td_body_")!=-1) {
        if (shiftTD){
          tds[i].className = "checkbox-click";
        }else{
          tds[i].className = "td_body";
        }
    }
  }
  shiftTD=!shiftTD;
}
</script>

så skulle den virke med det samme ... jeg checker lige udvidelsen ...

<ole>

roenving >> pelkjaer har helt ret i, at klasse-skift skal undgåes, hvor det overhovedet kan. MS anbefaler det endda selv.
Det betyder en hel del ... også selvom maskinen kører 2 GHz eller mere. Det er sådan set helt uden betydning.
At det går langsomt, er blot et tegn på, at metoden bruger en bunke unødige resourcer, da den kræver, at hele dokumentet faktisk reformateres, hvergang en klasse skiftes.

Specielt hvis man laver lidt større JS-applikationer, er det et blandt mange performance-problemer. Et godt eksempel er dette spm:
  http://www.eksperten.dk/spm/344547
om et CMS, der crasher IE6 på en _meget_ hurtig maskine. Det skyldes med højglansforkromet statsgaranti dårlig performende kode  ;o)

/mvh
</bole>