Comment-Tree auf/zuklappen

27
Bug #
Priorität3 (🔹 Normal)
BereichCommenting
TitleComment-Tree auf/zuklappen
BeschreibungDas auf/zuklappen der Comments funktioniert nicht.

JavaScript Error:
Das Objekt unterstützt diese Eigenschaft oder Methode nicht.
Reported by @
 
Git Commit
Assigned to @ 19.08.2004 01:17
StatusResolved @ 27
zorg.ch
#44184 by @ 18.08.2004 19:20 - nach oben -
Das müsstest wohl du anschauen. Die Comments benötigen folgenden Script:

<script language="javascript" type="text/javascript">
function onoff(id) {
layer = document.getElementById("layer" + id)
image = "img" + id;
if(layer.style.visibility == "hidden") {
document.images[image].src = "/images/forum/minus.gif";
layer.style.display = "block";
layer.style.visibility = "visible";
} else {
document.images[image].src = "/images/forum/plus.gif";
layer.style.display = "none";
layer.style.visibility = "hidden";
}
}

function reply() {
location.hash = "reply";
document.commentform.text.focus();
}
</script>
[Illegaler Tag: script]
zorg.ch
#44185 by @ 18.08.2004 19:29 - nach oben -
das script ist vorhanden. der parameter wird korrekt übergeben, das habe ich mit einem alert geprüft. es ist also einen fehler im JS-code.
zorg.ch
#44193 by @ 18.08.2004 19:47 - nach oben -
Vielleicht wäre ein <a href="#" onClick=" noch hilfreich...
zorg.ch
#44198 by @ 18.08.2004 19:54 - nach oben -
wieso hilfreich? der js-code wird ja ausgeführt. aber der code hat einen fehler drin...
zorg.ch
#44227 by @ 19.08.2004 00:05, edited @ 19.08.2004 00:26 - nach oben -
Meine JS-Kenntnisse sind zwar ein bisschen eingerostet, aber auf dem ersten Blick sehe ich weder Syntax-Fehler oder einen ungültigen Aufruf eines Objects.

Auf dem zweiten Blick funktioniert die Sache auch (Mozi 1.7 / IE 5.0). Ich denke nicht, dass es am Code liegt. Vielleicht liegt es eher daran, dass hier features verwendet werden, welche auf einem älteren oder exotischen Browser nicht wirklich funktionieren. Mit was hast du das ganze eigentlich gedebugt?
zorg.ch
#44238 by @ 19.08.2004 00:32 - nach oben -
der mozi macht keine fehlermeldung. da funktionierts einfach nicht. der ie machte ne JS-fehlermeldung:

zeile 11:
Das Objekt unterstützt diese Eigenschaft oder Methode nicht.

Die Funktion wird also aufgerufen. Das Bestätigt auch eine Überprüfung mit alert(id). die id wird auch korrekt übergeben.

zeile 11 ist diese hier (falls von file-anfang gezählt wird):
layer = document.getElementById("layer" + id)

oder diese hier (falls von script-anfang gezählt wird):
layer.style.visibility = "hidden";

ich glaube es wird vom file-anfang gezählt. bin aber nicht sicher.
zorg.ch
#44246 by @ 19.08.2004 00:57, edited @ 19.08.2004 00:59 - nach oben -
Soweit ich weiss, wird immer vom file-anfang gezählt. Der Fehler ist also in der folgenden Zeile zu suchen:

layer = document.getElementById("layer" + id)

mögliche Fehlerquellen:

1) Am Ende der Zeile fehlt ein Strichpunkt. Normalerweise ist das bei JS egal (nicht wie bei C), aber ich könnte mir denken, dass es auch Ausnahmen geben könnte.

2) getElementById kann das gesuchte Element nicht finden. Vielleicht wurde keinem Element die ensprechender ID zugewiesen (z.B. nur "layer" statt "layer1" )

3) IE schtinkt ussem muul

Hier noch mein Test-html

zorg.ch
#44256 by @ 19.08.2004 01:18 - nach oben -
2. war das problem. das hat mal funktioniert, hab aber wieder was geändert. drum hats die id nicht mehr korrekt zugewiesen.

zu 3) mozi stinkt no viel meh us em muul. de macht nid mol e fehlermeldig.
zorg.ch
#44259 by @ 19.08.2004 01:32 - nach oben -
Muesch bim Mozi/Firefox mol "javascript:" igeh id location bar... denn chunsch e ganz e schöns js-debugging-fenster über :-)
zorg.ch
#44262 by @ 19.08.2004 01:34 - nach oben -
mues me au no wüsse... ;)
zorg.ch
#44267 by @ 19.08.2004 01:38 - nach oben -
Isch ebe es Superuser Feature, nünt für so DAU's wie du ;-)
zorg.ch
#44279 by @ 19.08.2004 09:57 - nach oben -
lamber häsch am biko adä codingsession d'webdeveloper extension im firefox nöd zeigät ?
zorg.ch
#44284 by @ 19.08.2004 11:04 - nach oben -
Kurz scho, aber er isch halt en grobe IE-Junkie...
Additional posts
zorg.ch
#44285 by @ 19.08.2004 11:05 - nach oben -
nei, het er nid. bruch i au nid. de firefox bruch i nur, zum luege, öbs au dört lauft. und susch bin i ganz z friede mit mim ie. so hets wenigstens no eine, wo lueget, dass s layout au mit em ie no lauft. und im ie han i defür d zend tools.
Additional posts
zorg.ch
#44290 by @ 19.08.2004 11:16 - nach oben -
3) Stimmt so nicht. Der Mozi hat die besseren Dev-Funktionen als IE: JS Console, JS Debugger und einen DOM-Inspector. Das alles hat ein IE nicht. Die Fehlermeldung wird nur für die DAUs versteckt.
zorg.ch
#44294 by @ 19.08.2004 11:34 - nach oben -
ok, han i nid gwüsst. aber isch denn de mozi en dau-browser, dass er das versteckt?
zorg.ch
#44297 by @ 19.08.2004 11:36 - nach oben -
leider besteht die breite masse an usern aus DAU's
zorg.ch
#44298 by @ 19.08.2004 11:37 - nach oben -
schon. aber hab immer gedacht, dass der firefox ein geek-browser ist, der sich von der masse absetzen will.
zorg.ch
#44299 by @ 19.08.2004 11:45, edited @ 19.08.2004 11:45 - nach oben -
Mozilla war schon seit immer für die Masse gedacht.
Additional posts