Discussion :

[tintin72]

Bonjour,

Tout est dans le titre:

Code :

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

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
var pressed;
 
function keyUp(event)
{
    switch(event.keyCode)
    {
      case 8:
	pressed = false;
      break;
    }
}
 
function keyPress(event) 
{ 
    switch(event.keyCode)
    {
      case 8:
	if(pressed === true)
	{
	  var event = event || window.event; 
	  if(event.preventDefault) //FF
	    event.preventDefault();
 
	  event.returnValue = false; //IE ne marche pas
	}
	else
	  pressed = true;
      break;
    }
} 
 
var textarea = document.getElementById('text'); 
 
if(textarea.addEventListener) //FF
{
  textarea.addEventListener("keypress", keyPress, false);
  textarea.addEventListener("keyup", keyUp, false);
}
else if(textarea.attachEvent) //IE
{
  textarea.attachEvent("onkeypress", keyPress);
  textarea.attachEvent("onkeyup", keyUp);
}

Le code fonctionne très bien sous FF mais pas sous IE où le event.returnValue = false; semble n'avoir aucun effet.

Quelqu'un pourrait m'aider ?

Merci.

[Bovino]

preventDefault

[tintin72]

bah non, prevenDefault c'est pour FF.
Une idée ?

[Invité]

tu a omis un else

Code :

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

1
2
3
4
5
6
7
8
 
if(event.preventDefault){ //FF
 
 event.preventDefault();
}
 else{
  event.returnValue = false; //IE ne marche pas
	}

[Bovino]

bah non, prevenDefault c'est pour FF.

Effectivement, l'habitude de proposer preventDefault lorsque je lis returnValue fonctionne pas...

Ton problème vient du fait que pour IE, le paramètre (et donc la propriété) event n'existe pas

Code :

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

1
2
function keyUp(e){
    var event = e || window.event;

[bigboomshakala]

tu a omis un else

Code :

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

1
2
3
4
5
6
7
8
 
if(event.preventDefault){ //FF
 
 event.preventDefault();
}
 else{
  event.returnValue = false; //IE ne marche pas
	}

ça n'a aucune incidence.
sous IE ça met la propriété à false
sous FF ça crée une propriété returnValue avec la valeur false

[tintin72]

Ton problème vient du fait que pour IE, le paramètre (et donc la propriété) event n'existe pas
Code :

function keyUp(e){
var event = e || window.event;

J'ai modifié mon code mais j'ai toujours le même problème
avec IE.
Une idée ?

[Bovino]

Tu as essayé de le coupler avec cancelBubble ?

[tintin72]

Oui, mais ça marche toujours pas.

Voilà le code au cas où j'aurais zappé quelque chose ?

Code :

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

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
var pressed;
 
function keyUp(e)
{
    var event = e || window.event;
    switch(event.keyCode)
    {
      case 8:
	pressed = false;
      break;
    }
}
 
function keyPress(e) 
{ 
    var event = e || window.event;
    switch(event.keyCode)
    {
      case 8:
	if(pressed === true)
	{
	  var event = event || window.event; 
	  if(event.preventDefault) //FF
	    event.preventDefault();
 
	  event.returnValue = false; //IE ne marche pas
          event.cancelBubble();
	}
	else
	  pressed = true;
      break;
    }
} 
 
var textarea = document.getElementById('text'); 
 
if(textarea.addEventListener) //FF
{
  textarea.addEventListener("keypress", keyPress, false);
  textarea.addEventListener("keyup", keyUp, false);
}
else if(textarea.attachEvent) //IE
{
  textarea.attachEvent("onkeypress", keyPress);
  textarea.attachEvent("onkeyup", keyUp);
}

Une idée ?

[Bovino]

Tout d'abord, à ta place, je renommerais les fonctions keyUp et keyPress... Je sais que JavaScript est sensible à la casse, mais on est jamais trop prudent...

Ensuite :

Code :

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

event.cancelBubble();

Code :

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

event.cancelBubble = true;

Enfin, tu as une ligne inutile :

Code :

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

1
2
3
4
5
6
7
8
switch(event.keyCode)
{
  case 8:
    if(pressed === true)
    {
	  var event = event || window.event; 
	  if(event.preventDefault) //FF
	    event.preventDefault();

[tintin72]

J'ai renommé les 2 fonction (mykeyup et mykeypress), corrigé cancelBubble et
enlevé la ligne inutile (c'était un mauvais copié collé).
Mais toujours le même problème.

Je comprend pas.

[Bovino]

J'ai pas utilisé ces événements depuis quelque temps, mais en regardant ton code, il pourrait s'agir d'un déclenchement différent de keyup et keypress selon IE et FF.
Essaye déjà de voir si les événements sont bien déclenchés dans le même ordre

Code :

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

1
2
3
4
5
6
function keyPress(){
alert('événement keypress');
}
function keyUp(){
alert('événement keyup');
}

[tintin72]

L'alert keypress se déclenche lorsque j'appuie sur une touche, par contre
une fois que j'ai appuyé sur OK je n'ai pas l'alert keyup.
C'est normal ?

[tintin72]

Arf, je crois que j'ai trouvé.
Il faut initialiser la variable keypress à false dès sa déclaration:

Code :

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

var pressed = false;

Maintenant ça fonctionne.

Je vais quand même faire quelques tests supplémentaires au cas où.

[tintin72]

Rectification. En fait l'initialisation de pressed n'y change rien.
En fait IE réagit à keydown et pas à keypress.
Donc le code suivant semble bien fonctionner:

Code :

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

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
var pressed;
 
function keyUp(e) //IE + FF
{
    var event = e || window.event;
    switch(event.keyCode)
    {
      case 8:
	pressed = false;
	break;
    }
}
 
function keyDown(e) //IE
{ 
    var event = e || window.event;
    switch(event.keyCode)
    {
      case 8:
	if(pressed === true)
	{
	  event.returnValue = false;
	}
	else
	  pressed = true;
      break;
    }
} 
 
function keyPress(event) //FF
{ 
    switch(event.keyCode)
    {
      case 8:
	if(pressed === true)
	{
	  if(event.preventDefault)
	    event.preventDefault();
	}
	else
	  pressed = true;
      break;
    }
} 
 
var textarea = document.getElementById('text'); 
 
if(textarea.addEventListener) //FF
{
  textarea.addEventListener("keypress", keyPress, false);
  textarea.addEventListener("keyup", keyUp, false);
}
else if(textarea.attachEvent) //IE
{
  textarea.attachEvent("onkeyup", keyUp);
  textarea.attachEvent("onkeydown", keyDown);
}

Mais encore une fois quelques tests s'imposent avant d'être sûr.