JS: funktion definieren - fehler

hallo,

nun wollte ich mir doch ne art “gallerie” basteln, aber (wie immer, bei mir) mangelt es am Javascript…

die funtkion view hab ich so definiert:

function view (im) { imsr = '<p onclick="close();" style="font-size: 7px;">Close (X)</p><img src="http://ich.bplaced.net/img/' + im + '">'; document.GetElementById('img').innerHTML = imsr; document.GetElementById('img').style.visibility = 'visible'; }

nun kommt aber (beim FF) immer, wenn ich auf das bild (welches die funktion view ausführen soll) anklicke der Fehler:[quote]Fehler: view is not defined
Quelldatei: http://bcg.bplaced.net/aktuell[/quote]

also ist meine frage logischerweise: was hab ich falsch gemacht?

mfg Joey

Wo im Quelltext hast du die Funktion positioniert? Die muss vor dem Aufruf stehen - sont funktionierts nicht…

document.GetElementById('img').innerHTML = imsr; document.GetElementById('img').style.visibility = 'visible';

Es muss getElementById heißen, js ist case-sensitiv…

@thorr: :ps: wie bitte?? glaubst du das weiss ich nicht^^ (is ja wie in php… (@scram: ich mein nur das mit der positionierung))
ja vor dem aufruf ist es jeden falls schon …thx

@jazzon: … ich seh da einfach keinen unterschied, … nur die anstände, und das wirds kaum sein :unamused:

mfg Joey

/@jazzon: habs trozdem mal versucht, und wie erwartet, tuts das selbe wie vorher…

hast das get klein geschrieben?

uups!
beim zweiten anschauen hab ich garnicht mehr so genau geschaut, welches jetzt gehen sollte, und hab das im code feld genommen…

naja, vllt hats nochn anderen fehler, denn es tut immer noch nicht…

hier das ganze js:

[code][/code]

mfg Joey

Deine Javascript gehört in HEADER und nicht in den BODY

es heißt

type="text/javascript"

@uro: nee wiso denn?
dass es sicherlich geladen ist, beim aufruf, da stimme ich dir schon zu, aber… man kanns hin tun, wo man will^^

@michi7x7: ja, es geht… aber ich hab immer gedacht, dass es language heisst… aber anscheinend wegen dem text/ nicht Danke :hail:

mfg Joey

früher hieß es mal language, heute gilt das als veraltet, man sollte nur noch type verwenden.

ach ich hab eben gedacht, das wär so, dass man language verwenden sollte, aus irgendeinem grund…

mfg Joey