Hallo!

ich benutze folgenden JavaScriptCode auf meiner Seite:

Code :

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72

<script type="text/javascript" language="Javascript">    function right(e) {    if (navigator.appName == 'Netscape' &&    (e.which == 3 || e.which == 2))    return false;    else if (navigator.appName == 'Microsoft Internet Explorer' &&    (event.button == 2 || event.button == 3)) {    alert("fuck iexplorer, get FIREFOX!");    return false;    }    return true;    }    document.onmousedown=right;    if (document.layers) window.captureEvents(Event.MOUSEDOWN);    window.onmousedown=right; document.onmousedown = nrc; if (document.layers) window.captureEvents(Event.MOUSEDOWN); if (bNS && bV<5) window.onmousedown = nrc; function MM_preloadImages() {   var d=document; if(d.images){ if(!d.MM_p) d.MM_p=new Array();     var i,j=d.MM_p.length,a=MM_preloadImages.arguments; for(i=0; i<a.length; i++)     if (a[i].indexOf("#")!=0){ d.MM_p[j]=new Image; d.MM_p[j++].src=a[i];}} }   function MM_preloadImages() {   var d=document; if(d.images){ if(!d.MM_p) d.MM_p=new Array();     var i,j=d.MM_p.length,a=MM_preloadImages.arguments; for(i=0; i<a.length; i++)     if (a[i].indexOf("#")!=0){ d.MM_p[j]=new Image; d.MM_p[j++].src=a[i];}} }   function MM_findObj(n, d) {   var p,i,x;  if(!d) d=document; if((p=n.indexOf("?"))>0&&parent.frames.length) {     d=parent.frames[n.substring(p+1)].document; n=n.substring(0,p);}   if(!(x=d[n])&&d.all) x=d.all[n]; for (i=0;!x&&i<d.forms.length;i++) x=d.forms[i][n];   for(i=0;!x&&d.layers&&i<d.layers.length;i++) x=MM_findObj(n,d.layers[i].document);   if(!x && d.getElementById) x=d.getElementById(n); return x; }   function MM_nbGroup(event, grpName) {   var i,img,nbArr,args=MM_nbGroup.arguments;   if (event == "init" && args.length > 2) {     if ((img = MM_findObj(args[2])) != null && !img.MM_init) {       img.MM_init = true; img.MM_up = args[3]; img.MM_dn = img.src;       if ((nbArr = document[grpName]) == null) nbArr = document[grpName] = new Array();       nbArr[nbArr.length] = img;       for (i=4; i < args.length-1; i+=2) if ((img = MM_findObj(args[i])) != null) {         if (!img.MM_up) img.MM_up = img.src;         img.src = img.MM_dn = args[i+1];         nbArr[nbArr.length] = img;     } }   } else if (event == "over") {     document.MM_nbOver = nbArr = new Array();     for (i=1; i < args.length-1; i+=3) if ((img = MM_findObj(args[i])) != null) {       if (!img.MM_up) img.MM_up = img.src;       img.src = (img.MM_dn && args[i+2]) ? args[i+2] : ((args[i+1])? args[i+1] : img.MM_up);       nbArr[nbArr.length] = img;     }   } else if (event == "out" ) {     for (i=0; i < document.MM_nbOver.length; i++) {       img = document.MM_nbOver[i]; img.src = (img.MM_dn) ? img.MM_dn : img.MM_up; }   } else if (event == "down") {     nbArr = document[grpName];     if (nbArr)       for (i=0; i < nbArr.length; i++) { img=nbArr[i]; img.src = img.MM_up; img.MM_dn = 0; }     document[grpName] = nbArr = new Array();     for (i=2; i < args.length-1; i+=2) if ((img = MM_findObj(args[i])) != null) {       if (!img.MM_up) img.MM_up = img.src;       img.src = img.MM_dn = (args[i+1])? args[i+1] : img.MM_up;       nbArr[nbArr.length] = img;   } } } </script>

Um diesen Teil des Codes valide zu machen soll ich folgendes vor und nach dem Code einfügen:


<!--

<![CDATA[

und


]]>

//-->

Wenn ich das jetzt einfüge ist mein Code zwar einigermassen valide (es kommen keine Errors mehr nur noch Warnungen), aber mein JavaScript funktioniert nicht mehr!

Woran kann das liegen?

Danke für jede Hilfe!

Viele Grüsse
orffyre

Füge folgendes ein:

Code :

1 2 3 4 5 6 7

<script type="text/javascript" language="Javascript"> /* <![CDATA[ */   ...   /* ]]> */ </script>

Eine andere Möglichkeit wäre, das Javascript in eine externe Datei auszulagern.

Danke suye! Wie es aussieht funktioniert es jetzt! Aber was ist an dieser Schreibweise jetzt so anders als bei meiner!?

Das language-Attribut muss noch weg.

Dann noch eine Leselektüre: Ian Hickson: Sending XHTML as text/html Considered Harmful.