Externe .JS Datei aufbauen.

OJMD

Hallo zusammen,

ich überarbeite gerade meine Website und möchte gern alle meine Javascript Funktionen in eine Externe Datei schreiben und diese einfach nur in die jeweilige Seite einbinden.

Kann mir jemand sagen wo der Fehler hier in meinem Versuchsaufbau liegt?

Es soll das Untermenü eingefadet werden.

Index.html

PHP

<html>


<head>
	<script type="text/javascript" src="scripte/Script.js"></script>
</head>


<body>  
	<div id="MENU1" onmouseover='fading.ON("SUBMENU1");'>Home</div>




<div id="SUBMENU1" style="display:none;border:1px solid #000;background-color:#aaa">TEST TEST TEST</div>
 
</body>
</html>

Alles anzeigen

Script.js

PHP

var fading = {
	ON: function(obj){
		var step = 0;
		if (step <= 100){


		document.getElementByid(obj).style.display="block";
		document.getElementByid(obj).style.opacity = step;
		document.getElementByid(obj).style.filter = "alpha(opacity=" + step + ")"; // IE

		step++;
		setTimeout(fading.ON, 100);
		}
	},

	OFF: function(){

	},
}

Alles anzeigen

Ich sage schon mal vielen Dank.

Gruß OJMD

Pion

Jetzt geht es nicht und ohne externe Winbindung ging es?

Adesse falsch^^?

OJMD

Hallo Pion,

ich hab es hin bekommen, ein Komma war zuviel. (hinter der vorletzten Geschweiften Klammer)

Woran ich jetzt aber gerade Scheitere ist warum er nicht fadet, das müsste doch so eigentlich funktionieren, oder?

Code

var fading = {


	ON: function(obj,step){
		if (step <= 100){
			document.getElementById(obj).style.display="block";
			document.getElementById(obj).style.opacity = step/100;
			document.getElementById(obj).style.filter = "alpha(opacity=" + step + ")"; // IE
			step=step+1;
			setTimeout(fading.ON(obj,step), 100);
		}
	},

	OFF: function(obj){
		document.getElementById(obj).style.display="none";
	}
}

Alles anzeigen

Er weigert sich aber und blendet es sofort ein.

Gruß OJMD

jojo87

du kannst meines wissens nach bei settimeout keine argumente übergeben.
zudem muss die funktion als string übergeben werden.
versuchs mal so:

Code

setTimeout('fading.ON('+obj+')', 100);

Schöner wärs, wenn du das zu fadende div als Objektvariable setzt und dann im onload ein click-event setzt.

OJMD

Hallo jojo87,

ich habe mehrere Javascript ".js" Dateien angeschaut, in denen wird der Ausdruck nicht in Klammern gesetzt. Die Funktion funktioniert ja auch, sie wird 100 mal aufgerufen (geprüft mit Alert(step);) das opacity wird auch geändert. Das einzige was nicht geht ist die Verzögerung, egal was ich reinschreibe 1, 500, 5000000 es erscheid sofort.

Gruß OJMD

OJMD

OK hab es jetzt hin bekommen und gleich ein neues Problem. Der onmousover wird schon aktiviert ob wohl ich noch auch dem div bin. Jetzt Flimmert mein Menü wenn ich über die Links mit der Maus fahre und das event onmouseout wenn ich das Menü verlasse funktioniert jetzt auch nicht immer richtig.

Kann mir jemand sagen warum?

Code

var fading = {  
st: setTimeout('null',1),
speed: 2,
todo: 1,
id:'',
step:0,

	STARTON: function(obj){
       		fading.id = obj 
       		fading.ON() 
       	},
	ON: function(){  
		if (fading.step < 100){
			document.getElementById(fading.id).style.display="block"
			document.getElementById(fading.id).style.opacity = fading.step/100
			document.getElementById(fading.id).style.filter = "alpha(opacity=" + fading.step + ")"; // IE
			fading.step+=2;
		        fading.st = setTimeout('fading.ON()',(fading.speed))
		}else{
			clearTimeout(fading.st)
			fading.st = setTimeout('null',1)  


		}
	},

	OFF: function(obj){ 
		document.getElementById(obj).style.display="none"
		fading.step = 0
	},

	SHOWOUT: function(obj){

       		if(fading.todo == 1){ 
   	      		document.getElementById(obj).style.display="block"
   	      		fading.todo = 0
      		}else{
   	       		document.getElementById(obj).style.display="none"
   			fading.todo = 1
      		}
      }
}

Alles anzeigen

Tags