script compatible firefox

[ddilag]

Comment faire pour rendre compatible le script du menu déroulant de mon site ? il fonctionne sous IE mais pas sous firefox.

http://www.dinokc.com

merci d'avance

[calimo]

1. Poser la question dans la rubrique appropriée : développement web
2. Je n'arrive même pas à entrer. Je n'ai pas trop de temps maintenant, je verrai ça plus tard si j'y pense et que le sujet n'es pas perdu dans les abysses de cette rubrique

[Andevros]

Même problème pour moi.
Aprés la première page .... -> Frame introuvable ...etc. --> écran gris.

Firefoc 1.0.4

[Andevros]

C'est toujours moi.

Mauvaise nouvelle.
Ton site n'est compatible qu'avec IE.
En effet avec Opèra pas moyen d'entrer.
Il y a du travail en perseptive.

[calimo]

Je viens de comprendre pourquoi on ne peut pas rentrer... je dois avouer que je n'ai jamais vu ça ! Ça vient de la fonction function redirectPage()

1. C'est bête de réserver l'accès à ton site à ceux qui ont le javascript et qui naviguent à la souris. 10% des internautes n'ont pas le javascript, et je ne sais pas combien naviguent au clavirer. Aucune de ces deux catégories de personnes ne peut rentrer.

   Code : Tout sélectionner

   <INPUT onclick=redirectPage() type=button value="ENTREZ & GO" style="background-color:#FF9900;">

   Il faut donc pouvoir cliquer sur le bouton et avoir JS activé. Si je puis me permettre, tu devrais au moins changer pour mettre

   Code : Tout sélectionner

   <form action="1frames.html"><input onclick="redirectPage(); return false;" type="submit" value="ENTREZ & GO" style="background-color:#FF9900;"></form>

   Normalement ça ne devrait rien changer à l'aspect du bouton, mais c'est un bouton de soumission du formulaire et donc en soumettant, ceux qui n'ont ni JS ni souris peuvent passer. Le mieux serait encore de faire un vrai lien, mais bon...
2. J'ai gardé le meilleur pour la fin... la fonction redirectPage() :

   Code : Tout sélectionner

   <SCRIPT language=JavaScript>
   function redirectPage() {
    if ((screen.width == 640) && (screen.height == 480)) 
       {
   	open("1frames.html","V02","LEFT=0,Top=0,WIDTH=630,HEIGHT=389,status=yes,toolbar=yes,scrollbars");
   	}
   else if ((screen.width == 800) && (screen.height == 600)) 
   	{
   	open("1frames.html","VO2","LEFT=0,Top=0,WIDTH=780,HEIGHT=495,status=yes,toolbar=yes,scrollbars");
   	}
   else if ((screen.width == 1024) && (screen.height == 768)) 
   	{
   	open("1frames.html","V02","LEFT=0,Top=0,WIDTH=1014,HEIGHT=651,status=yes,toolbar=yes,scrollbars");
   	}
   }
   </SCRIPT>

   Je dois avouer que je n'ai jamais vu un truc pareil
   Eh oui, il faut avoir une des 3 résolutions mentionnées pour pouvoir rentrer ! Si on a une autre résolution, on ne rentre pas. C'est totalement débile

Conclusion, je ne sais pas où tu as pris cette fonction, supprime-là purement et simplement il n'y a rien à en retirer, tout à jeter. Et à la place du bouton, remplace

Code : Tout sélectionner

<FORM><INPUT onclick=redirectPage() type=button value="ENTREZ & GO" style="background-color:#FF9900;"> 
</FORM>

par

Code : Tout sélectionner

<a href="1frames.html" style="background-color:#FF9900; border:1px solid #555">ENTREZ & GO</a>

Ça devrait avoir la même apparence (au pire on peut toujours appliquer un peu de CSS), et tout le monde poura rentrer sans discriminations

[ddilag]

Merci de toutes vos remarques, j'ai modifié le script en conséquence.

http://www.dinokc.com

Mais j'ai toujours le problème avec firefox sur mon menu déroulant. j'ai mis en ligne la page incriminée.

http://www.dinokc.com/menu.html

merci d'avance.

[calimo]

Là encore tu as des détections de navigateur pas terribles...

Code : Tout sélectionner

var n = (document.layers) ? 1:0;
var ie = (document.all) ? 1:0;
var browser=((n || ie) && parseInt(navigator.appVersion)>=4)

et plus loin

Code : Tout sélectionner

if(browser)

sans le moindre help.
Évidemment, Firefox ne connaît ni document.layers, ni document.all, qui sont propriétaires respectivement de Netscape (4.x) et de IE.

C'est bizarre parce que je n'ai pas vu plus loin de if(ie) ou if(n), donc ça veut dire que ce bout de code est juste là pour s'assurer qu'on a affaire à IE ou Netscape 4.x dans une version >=4

Supprime ces conditions

Cela dit je ne pense pas que ça suffira parce que ça ne passe pas non plus dans Opera (qui implémente aussi le document.all d'IE)

[ddilag]

Les modifications n'ont pas suffit.[/i]

[ddilag]

Et en plus ça ne marche plus avec IE. Je fais marche arrière en attendant mieux.[/i]

[Benoit]

A priori, c'est la fonction makeMenu qui doit être réécrite pour utiliser le méthode standard document.getElementById() au lieu de document.all et son alternative pour Netscape 4. Mais peut-être que tu irais plus vite en cherchant un autre script de menu plus récent en fait, parce qu'il semble assez "siècle dernier"

[ddilag]

je pense que tu as raison. je vais chercher un nouveau script.