xslc.js, SetFocus() et IE6

[Jeweller]

Hello!
J'ai une petite remarque, la routine SetFocus de xslc.js attribue le focus à un élément. Sous IE6 (et dans certains cas?, dans mon contexte d'utilisation en l'occurence) celle-ci fait "planter" le navigateur.
En fait si la zone en question ne peut pas obtenir le focus, cela ne fonctionne forcément pas, mais le soucis c'est que IE part en vrille et il faut le relancer.
Attention donc en utilisant SetFocus à tester avant si le focus est possible!

Je propose donc que cette fonction face les tests necessaires avant, un truc du genre:

Code :

1
2
3
 
   if (obj.type != 'text') return false;
   if (obj.disabled || obj.readonly)  return false;

avant le obj.focus();

Michaël

[powerlog]

Merci.
Très bonne info. Il faudrait prendre en compte également le bouton ou la case à cocher.

[powerlog]

je voulais dire "le bouton radio"

[Jeweller]

Ben j'y ai pensé au bouton radio, mais j'ai pas testé et je suis pas sur que cela pose pas le même soucis, car le check ou le radio ne peuvent à peine plus prendre le focus qu'une zone read only... à tester donc!

[Nicolas.Cogi]

Le code actuel xslc.js:

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
function SetFocus(ID, Begin, Select) {
	var obj, txt;
	if (Begin == null)
		Begin = true;
	if (Select == null)
		Select = false;
	if (obj = getObject(ID))
	{
		if (isVisible(obj) == false || isDisabled(obj))
			return;
		obj.focus();
		if (Begin == false)
		{
			if (obj.createTextRange)
			{
				txt = obj.createTextRange();
				txt.collapse(false);
				txt.select();
			}
		}
		if (Select == true)
		{
			obj.select();
		}
	}
}

Test fait sur isVisible() et isDisable() avant le focus, pour les memes raisons.

Est-ce que tu as testé l'attribut readonly sur d'autre navigateurs ?

[Jeweller]

Oui, j'ai vu dans la beta c'est un peu mieux, mais pas suffisant à mon avis.
IsVisible renverra vrai si le champ est de type hidden par exemple et donc le SetFocus sera fait quand mêm et ca plantera (c'etait mon cas) !
Par contre je n'ai pas testé le readonly sur d'autres navigateurs. FF apparement cela le dérange moins. Mais bon, dès lors que la zone ne peut avoir le focus, il n'y a pas lieu d'essayer de le lui donner.

Michael