fel i firefox!

hej.

jag har precis slängt upp http://www.komab.se/ och får ett lurigt fel i firefox!

när man går in under en produkt, så slutar top-menyn att fungera och kan för mitt liv inte förstå varför.. det fungerar bra i IE.
någon?
snabbt svar vore toppen då sidan "lanserades" i fredags...

Allt ser likadant ut för mig i både firefox och IE.

Vilken version av firefox har du då det krånglar?

red/ nu förstår jag vad du menade. miss av mig.

Den omedelbara skillnaden är: onFocus="this.blur()"

Produktsidorna:

<link onFocus="this.blur()" href="komab.css" rel="stylesheet" type="text/css">

andra sidor:

<link href="komab.css" rel="stylesheet" type="text/css">

[red]<link [b]onFocus="this.blur()"[/b] href="komab.css" rel="stylesheet" type="text/css">[/red]

ser lite konstigt ut :OO

mv icaaq

Jag kan inte peka ut felet, men jag kan tyvärr konstatera att menyn är ganska dåligt kodad. Jag listr här några fel, även om jag inte ser något direkt samband med ff-problemen.

if (document.images) {
	omon=new Image();
	omon.src="/img/om_oss-on.gif";
	omoff=new Image();
	omoff.src="/img/om_oss.gif";

	ftgon=new Image();
	ftgon.src="/img/foretag-on.gif";
	ftgoff=new Image();
	ftgoff.src="/img/foretag.gif";

	kamon=new Image();
	kamon.src="/img/kampanjer-on.gif";
	kamoff=new Image();
	kamoff.src="/img/kampanjer.gif";

	buton=new Image();
	buton.src="/img/butiker-on.gif";
	butoff=new Image();
	butoff.src="/img/butiker.gif";

	konton=new Image();
	konton.src="/img/kontakta_oss-on.gif";
	kontoff=new Image();
	kontoff.src="/img/kontakta_oss.gif";
}

1. Använd new Image() inte new Image
2. Ange absoluta sökvägar: Alltså t ex omon.src="/img/om_oss-on.gif";

Sedan har vi ditt överanvändande av eval :OO

function changeStyle(Id,Property,Value){
	if (document.all){
		eval('document.all[Id].style.'+Property+'="'+Value+'"');
	} else if (document.layers){
		eval('document.layers[Id].'+Property+'="'+Value+'"');
	} else if (document.getElementById){
		eval('document.getElementById(Id).style.'+Property+'="'+Value+'"');
	}
}

blir med fördel:

function changeStyle(Id,Property,Value){
	document.getElementById(Id).style[Property]=Value);
}

Notera tre saker:

1. Särbehandla inte ie! (doument.getElementById stöds sedan länge)
2. Skippa NS4-koden! (document.layers funkar ändå inte som du vill i detta fall)
3. Jag tror att FF inte hittar variabeln Id när du lägger den i eval, det tror jag är felet.

I övrigt finns det mycket som skulle kunna förbättras, men det tänker jag inte lägga min energi på nu.

nitro2k01
hittat på webforum, så jag får skylla på mina bristfälliga javascript-kunskaper.
tack för kodexemplet dock, ska lägga lite energi på att förbättra koden.. senare. =)

isaaq / frankof
där kan nog skon klämma. jag kikar på det strax. tack!
anledningen till att det ser ut sådär är att jag gjorde en "ersätt alla " href" med " onFocus="this.blur()" href". alltså slarv. =)

onfocus="this.blur()" är inte speciellt bra för användarvänlighet då det hämmar dem som använder tangentbordet istället för musen.

Om du vill få bort den "fula" ramen kan du ändra det i stilmallen. Åtminstone för Firefox. Jag skulle faktiskt försöka göra det lättare för tangentbordsanvändare istället för svårare men det kanske bara är jag. :)

a:focus, a:active {
 -moz-outline:none;
 background:Highlight;
 color:HighlightText;
 text-decoration:none; 
}

zcorpan
det kanske kan vara bra, men jag ids inte göra om - till nästa site, jag lovar. =)

efter lite slit i håret så hittade jag en lösning iallafall..
jag hade uppenbarligen en "</td></tr></table>" för mycket
på slutet. känsliga jävel ;)

Sånna fel man kan undvika om man validerar... ;)