nicht angemeldet
Hallo,
»block« funktioniert wie gesagt für den MSIE. Das ist der interne display-Wert für Zellen (siehe zellenelementknoten.currentStyle.display). Die möglichen display-Werte stehen auch hier: http://msdn.microsoft.com/workshop/author/dhtml/reference/properties/display.asp.
Darüber hinaus:
function appear(obj) {
if (document.getElementById && !document.all){
//Mozilla - funktioniert!
pre = 'document.getElementById("';
post = '").style';
}
if (document.getElementById && document.all){
//IE 5.5+ - funktioniert nicht!
pre = 'document.all.';
post = '.style';
}var myLayer = eval(pre + obj + post);
if (myLayer.display == "none") {
myLayer.display = "table-cell";
}
else {
myLayer.display = "none";
}
}
1. eval() ist unnötig und ineffizient. Bei getElementById brauchst du eval gar nicht (document.getElementById(obj)) und bei document.all auch nicht (document.all[obj], siehe http://selfhtml.teamone.de/javascript/objekte/all.htm#mit_namen_ansprechen).
2. Die Browserweiche ist widersinnig: IE 5.x+ kannst du ebenfalls über getElementById versorgen, daher ist die Weiche an der Stelle unnötig. Sie wäre höchstens nötig, wenn du auch IE 4 versorgen willst. Dann ist aber document.getElementById && document.all keine angemessene Bedingung.
3. Mit document.all lässt sich nicht zuverlässig MSIE erkennen. Opera kennt document.all auch (was in obigem Fall nicht schlimm ist, was aber eine Rolle spielt, wenn es darum geht, zu unterscheiden, wer table-cell und wer block bekommt).
Ich würde es so lösen:
function appear (id) {
if (document.getElementById) {
var myLayer = document.getElementById(id);
} /* else if (document.all) var myLayer = document.all[id]; */
if (window['display'+id]) { // in displayXXX wird der Status gespeichert (true=sichtbar/false=unsichtbar)
myLayer.style.display = 'none';
window['display'+id]=false;
} else {
if (document.uniqueID) {
myLayer.style.display = 'block';
} else {
myLayer.style.display = 'table-cell';
}
window['display'+id]=true;
}
}
Mit document.uniqueID soll MSIE 5.x+ identifiziert werden. Denkbar wäre auch meines Wissens document.all && !window.opera.
Der Fehler sollte nicht bei der Browser detection liegen, sondern in der 'if (myLayer.display...' zeile - IE meint 'Could not get the display property. Invalid argument.'
Das weißt du ja mittlerweile: Er erkennt »table-cell« nicht als gültigen Wert für display an.
Noch was: wenn ich "display" in der table-cell auf "table-cell" als Ausgangsposition setze statt auf "none", kann ich sie korrekt zum verschwinden bringen, der fehler kommt nur wenn display == none!
Klar, weil er style="display:table-cell" aus oben genannten Gründen vollkommen ignoriert. Somit liefert .style.display einen leeren String zurück. Spätestens beim Umschalten auf table-cell kommt aber der obige Fehler.
Mathias