Forum PHP.pl > [js] setAttribute nie działa w IE

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [js] setAttribute nie działa w IE

Forum PHP.pl > Forum > Po stronie przeglądarki > JavaScript

dyktek

16.08.2006, 10:17:07

Napisałem ostatnio kodzik w js, takie ostatnio popularne gwiazdki do głosowania, wynik działania poza IE widać tutaj problem polega właśnie w IE, nie wiem czemu ten kod nie działa pod tym badziewiem, wydaje mi się że problem tkwi w setAttribute, wie ktoś może jak to ominąć?

[HTML] pobierz, plaintext 
<html>
	<head>
		<script type="text/javascript">
 
		var g_clear = 'g_white.gif';
		var g_fill = 'g_green.gif';
		var ile = 5;
 
		var x = 29;
		var y = 29;
 
		function $( id ){
			return document.getElementById( id );
		}
 
		function generuj(){
			var iden = $( 'ocena' );
 
			for( i = 0; i < ile; i++ ){
 
			docBody = $( 'ocena' );
			element = document.createElement("img");
			element.id = i ;
			docBody.appendChild(element);
 
			incre = i + 1;
 
			element.setAttribute("src", g_clear);
			element.setAttribute("width", x);
			element.setAttribute("height", y);
 
			element.setAttribute("onMouseOver", "zmien('"+incre+"');");
 
 
			$('ocena').setAttribute("onMouseOut", "czysc();");	
 
			}
 
		}
 
		function zmien(nr){
			for(i = 0; i < nr; i++)
				$(i).setAttribute("src", g_fill);	
		}
 
		function czysc(){
			for( i = 0; i < ile; i++ )
				$(i).setAttribute("src", g_clear);	
		}
 
		</script>
	</head>
	<body onload="generuj();">
 
		<div id="ocena"></div>
 
	</body>
</html>
[HTML] pobierz, plaintext

właściwie to już sobie poradziłem

trzeba było zrobić taki cuś:

[HTML] pobierz, plaintext 
if(navigator.appName == 'Microsoft Internet Explorer'){
					element.setAttribute("onmouseover", function zaz(){ zmien(this.name); });
					element.setAttribute("onmouseout", function dez(){ czysc(); });
				} else {
					element.setAttribute("onMouseOver", "zmien('"+incre+"');");
					$('ocena').setAttribute("onMouseOut", "czysc();");	
				}
[HTML] pobierz, plaintext

Zajec

16.08.2006, 10:55:13

Konstrukcja:

Kod

element.setAttribute("onMouseOver", "zmien('"+incre+"');");

jest niepoprawna i działa chyba tylko w Operze. Nie można traktować wartości zdarzeń jako string!

To co zrobiłeś z wykrywaniem przeglądarki jest też złe. A skąd wiesz, że używana przeze mnie Opera nie przedstawia się akurat jako IE? Zaserwujesz mi kod dla IE, czego przecież nie chciałeś.

Kod, który powinien dzialać wszędzie:

Kod

element.onmouseover=function () { zmien(this.name); };
element.onmouseout=function () { czysc(); };

dyktek

16.08.2006, 11:12:39

poprawiłem i działa

tylko nie wiem dlaczego w operze nie mozna zaznaczyć ostatniej gwiazdki, wynik tutaj, dzięki za pomoc

Zajec

16.08.2006, 15:34:23

Cytat(dyktek @ 16.08.2006, 12:12 )

poprawiłem i działa

tylko nie wiem dlaczego w operze nie mozna zaznaczyć ostatniej gwiazdki, wynik tutaj, dzięki za pomoc

Opera głupieje, ponieważ Twoje oznaczenia "id" rozpoczynają się od cyfry. A to jest niedozwolone wg W3C.

phpion

29.10.2006, 16:49:56

Pozwalam sobie odswiezyc temat. Mam kod:

[HTML] pobierz, plaintext 
var cell8 = row.insertCell(8);
cell8.setAttribute('class', 'green');
[HTML] pobierz, plaintext

Pod Opera oraz FF dziala on poprawnie, natomiast na IE nie koloruje komorki tabeli (klasa green odpowiedzialna jest m.in. za zielone tlo). Wywolanie w ten sposob:

[HTML] pobierz, plaintext 
var cell8 = row.insertCell(8);
cell8.class = 'green';
[HTML] pobierz, plaintext

W IE zwraca blad skryptu (na pasku stanu), naotmiast w Operze i FF wogole nie dziala.
Pomoglby ktos w tym problemiku?

Zajec

29.10.2006, 23:14:55

Cytat(phpion.com @ 29.10.2006, 16:49:56 )

Pod Opera oraz FF dziala on poprawnie, natomiast na IE nie koloruje komorki tabeli

Eh, wybić tych od M$ :-|

Spróbuj

Kod

cell8.className = 'green';

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.