Discussion :

[Kalram]

Bonjour,

J'essaye de trouver l'origine d'un bug qui survient seulement sous Firefox 10, à priori lié à Javascript, mais j'aimerais avoir votre opinion.

Le site :
www.prix-portables.fr

Navigateurs touchés :
Firefox 10, 10.0.1, 10.0.2

En 2008, une touche d'ajax a été rajoutée au comparateur, permettant de rafraîchir la liste des résultats dynamiquement dès qu'une valeur est sélectionnée dans l'un des menus déroulants (événement "onChange")

Explication du bug :
- Sous les anciennes version de Firefox, Chrome ou IE, pas de problème la liste est correctement rafraîchie.
- À partir de Firefox 10, la liste de résultats n'est réellement actualisée qu'une fois sur deux, et à chaque sélection de valeur ou presque l'affichage vacille.

Constats :
En utilisant par exemple l'UserAgent de Chrome 11 sous Firefox 10, plus de problème.


- J'ai donc pensé à une éventuelle évolution du moteur Javascript de Firefox qui ne supporte plus une ancienne syntaxe, est-ce possible ?
- Autre piste, un problème de détection de navigateur : un article récent met en garde contre les scripts de détection de navigateus obselètes qui ne reconnaissent pas les numéros de version à deux chiffres, ne prenant en compte que le premier (FF10 serait donc reconnu comme FF1).
- La mise à jour des bibliothèques utilisées n'a rien résolu non plus, il s'agit de Prototype.js et dsHistory.js.

Voici les versions en ligne sur le site :
http://www.prix-portables.fr/js/dshistory.js
http://www.prix-portables.fr/js/prototype.js

Merci de m'avoir lu, et bonne journée à vous !

[Kalram]

En explorant la piste du script de détection de navigateur, je me dis que celui utilisé dans dsHistory.js est peut-être le coupable, cette bibliothèque n'étant plus mise à jour par son auteur depuis décembre 2008.

Est-ce que vous voyez quelque chose de suspect dans ce morceau de code ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
// we need a good browser detection library. these detections were kindly borrowed from the Prototype library
	var browser = (function() {
		var userAgent = window.navigator.userAgent;
		var isIE = !!(window.attachEvent && !window.opera && userAgent.indexOf('Opera') == -1);
 
		return {
			IE: isIE,
			IE6: isIE && userAgent.indexOf('MSIE 6') != -1,
			IE7: isIE && userAgent.indexOf('MSIE 7') != -1,
			Opera: !!window.opera && userAgent.indexOf('Opera') != -1,
			WebKit: userAgent.indexOf('AppleWebKit/') > -1,
			Gecko: userAgent.indexOf('Gecko') > -1 && userAgent.indexOf('KHTML') == -1
		};
	})();