IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

JavaScript Discussion :

Evénement input sur element select ne fonctionne pas


Sujet :

JavaScript

  1. #1
    Membre éclairé
    Homme Profil pro
    Ingénieur en électrotechnique retraité
    Inscrit en
    Décembre 2008
    Messages
    1 603
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 72
    Localisation : France, Bas Rhin (Alsace)

    Informations professionnelles :
    Activité : Ingénieur en électrotechnique retraité

    Informations forums :
    Inscription : Décembre 2008
    Messages : 1 603
    Points : 818
    Points
    818
    Par défaut Evénement input sur element select ne fonctionne pas
    Bonjour à tous,

    Dans la doc MDN, je lis "Les évènements déclenchés par <select> : input, change"

    Si je modifie la valeur d'un élément input, l'événement est déclenché mais ce n'est pas le cas si je change l'option d'un élément select, ni si j'y saisi les premiers caractères. Ligne 4, la console retourne bien tous les éléments et en particulier, les options de l'élément select.

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    10
    11
    	const cardForm	= document.querySelector(".cardForm")
    		, cardFields= cardForm.querySelectorAll("input, select, textarea")
    		;
    	console.log(cardFields);
    	for (let i=0, max=cardFields.length; i<max; i++)
    	{
    		cardFields[i].addEventListener('input', function(e)
    		{
    			alert(this.name);
    		},false);
    	}
    Page html générée
    Code html : 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
    <!DOCTYPE html>
    <html>
    	<head>
    		<meta charset="utf-8" />
    	</head>
    	<body>
    		<!-- J'ai supprimé quelques balises pour ne pas alourdir -->
    <form method="post" action="http://sirep.proginet.local/frontend/controllers/addressHandle.php" class="cardForm">
    	<div>
    		<label for="company">Client</label><input type="text" name="company" id="company" value="Nom" /><br/>
    		<label for="address1">Adresse 1</label><input type="text" name="address1" id="address1" value="avenue de San Severo" /><br/>
    		<label for="country">Pays</label><select name="countries" id="countries"><option></option><option value="AF">Afghanistan</option><option value="ZA">Afrique du Sud</option><option value="FR" selected="selected">France</option><option value="ZW">Zimbabwe</option></select><br/>
    		<label for="zip">Code postal</label><input type="text" name="zip" id="zip" value="01234" /><br/>
    		<label for="city">Ville</label><input type="text" name="city" id="city" value="Ma ville" /><br/>
    	</div>
    	<div>
    		<label for="phone">Téléphone</label><input type="text" name="phone" id="phone" value="01 23 45 67 89" /><br/>
    		<label for="note">Annotation</label><input type="text" name="note" id="note" value="" /><br/>
    	</div>
    </form>
     
     
     
     
    	</body>
    </html>

  2. #2
    Invité
    Invité(e)
    Par défaut
    Bonjour,

    1- avec les codes fournis, ça fonctionne.

    2- On peut aussi écrire :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    cardFields.forEach( function(cardField){
      cardField.addEventListener('input', function(e){
        console.log(this.name, this.value);
      });
    });

  3. #3
    Membre éclairé
    Homme Profil pro
    Ingénieur en électrotechnique retraité
    Inscrit en
    Décembre 2008
    Messages
    1 603
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 72
    Localisation : France, Bas Rhin (Alsace)

    Informations professionnelles :
    Activité : Ingénieur en électrotechnique retraité

    Informations forums :
    Inscription : Décembre 2008
    Messages : 1 603
    Points : 818
    Points
    818
    Par défaut
    Citation Envoyé par jreaux62 Voir le message
    Bonjour,
    1- avec les codes fournis, ça fonctionne.
    Il faut que je regarde pourquoi, ça ne fonctionne pas chez moi. En effet, lorsque je fais un copier/coller in extenso du fichier source et du code JS dans JSFiddle avec Chrome, tout fonctionne.
    EDIT:
    • Si j'enregistre la source dans un fichier de mon application et que j'affiche ce fichier dans mon navigateur, le problème persiste.
    • Il semblerait que ce soit un problème du navigateur (IE11) parce que si je fais l'essai dans un autre navigateur (Chrome), le fonctionnement est correct.
    • JSFiddle ne fonctionne pas avec IE.


    Citation Envoyé par jreaux62 Voir le message
    2- On peut aussi écrire :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    cardFields.forEach( function(cardField){
      cardField.addEventListener('input', function(e){
        console.log(this.name, this.value);
      });
    });
    Non, ça ne fonctionne pas parce que cardFields n'est pas un tableau mais une liste de nœuds, je crois. Le navigateur (IE11 m'indique une erreur : L'objet ne gère pas la méthode "forEach")

  4. #4
    Invité
    Invité(e)
    Par défaut
    Citation Envoyé par moimp Voir le message
    ...cardFields n'est pas un tableau mais une liste de nœuds...
    Sauf erreur, on a déjà eu cette discussion, non ?...

    Il suffit d'utiliser Array.from() :

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    cardFields = Array.from( cardForm.querySelectorAll("input, select, textarea") )
    Auquel cas .forEach() s'appliquera bien à un tableau (array).
    Compatible depuis I.E. 9.

  5. #5
    Membre éclairé
    Homme Profil pro
    Ingénieur en électrotechnique retraité
    Inscrit en
    Décembre 2008
    Messages
    1 603
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 72
    Localisation : France, Bas Rhin (Alsace)

    Informations professionnelles :
    Activité : Ingénieur en électrotechnique retraité

    Informations forums :
    Inscription : Décembre 2008
    Messages : 1 603
    Points : 818
    Points
    818
    Par défaut
    C'est vrai, nous en avons déjà parlé, mais non, il n'est pas géré pour la méthode .from. Voir le tableau des compatibilités MDN ici. C'est pour ça que je ne l'utilise pas.

  6. #6
    Invité
    Invité(e)
    Par défaut
    Oups...

  7. #7
    Expert confirmé Avatar de Toufik83
    Homme Profil pro
    Développeur informatique
    Inscrit en
    Janvier 2012
    Messages
    2 440
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 41
    Localisation : Suisse

    Informations professionnelles :
    Activité : Développeur informatique
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Janvier 2012
    Messages : 2 440
    Points : 4 942
    Points
    4 942
    Par défaut
    Salut,

    Il y'a une autre façon pour attacher l'événement "input" ou "change" tout en gardant la compatibilité avec IE
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
     
    for (let i = 0, max = cardFields.length; i < max; i++) {
          cardFields[i].addEventListener(cardFields[i].nodeName.toLowerCase() == 'input' ? 'input' : 'change'
                   , function(e) {//si c'est un input alors on attache l’événement input, sinon on attache  change
                         console.log("name :", this.name,', event :',e.type);
         },false);
    }
    comme on peut aussi parcourir le Nodelist "cardFields" avec deux manières :

    méthode 1 :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
     
    Array.prototype.slice.call(cardFields).forEach(function(element){
        console.log('element.name :',element.name);
    });
    Méthode 2 :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
     
    [...cardFields].forEach(element =>{
        console.log('element.name :',element.name);
    });
    Donc on peut attacher les événements avec les deux méthodes ci-dessus :

    Avec la méthode 1 :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
     
    Array.prototype.slice.call(cardFields).forEach(function(element) {
    	  element.addEventListener(element.nodeName.toLowerCase() == "input" ? "input" : 'change', function(e) {
    	    console.log("name :", this.name);
    	  }, false);
    });
    Avec la méthode 2 :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
     
    [...cardFields].forEach(element => {
    	  element.addEventListener(element.nodeName.toLowerCase() == "input" ? "input" : 'change', function(e) {
    	    console.log("name :", this.name);
    	  }, false);
    });

  8. #8
    Modérateur

    Avatar de NoSmoking
    Homme Profil pro
    Inscrit en
    Janvier 2011
    Messages
    17 098
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Isère (Rhône Alpes)

    Informations forums :
    Inscription : Janvier 2011
    Messages : 17 098
    Points : 44 675
    Points
    44 675
    Par défaut Remarques
    Bonjour,
    la syntaxe Destructuring assignment n'est pas prise en compte par IE, donc exit la méthode 2 [...cardFields].forEach(element =>{.

    Nota :
    la propriété nodeName comme tagName, d'un élément HTML renvoi une chaine en majuscule donc inutile de faire le test sur les minuscules
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    cardFields[i].nodeName == "INPUT" ? 'input' : 'change'

  9. #9
    Expert confirmé Avatar de Toufik83
    Homme Profil pro
    Développeur informatique
    Inscrit en
    Janvier 2012
    Messages
    2 440
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 41
    Localisation : Suisse

    Informations professionnelles :
    Activité : Développeur informatique
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Janvier 2012
    Messages : 2 440
    Points : 4 942
    Points
    4 942
    Par défaut
    Citation Envoyé par NoSmoking Voir le message
    Bonjour,
    la syntaxe Destructuring assignment n'est pas prise en compte par IE
    Ben si, je viens de tester la méthode 1 sous IE 11 et cela fonctionne, par-contre la méthode 2 n'est pas compatible avec IE mais compatible avec EDGE.

  10. #10
    Modérateur

    Avatar de NoSmoking
    Homme Profil pro
    Inscrit en
    Janvier 2011
    Messages
    17 098
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Isère (Rhône Alpes)

    Informations forums :
    Inscription : Janvier 2011
    Messages : 17 098
    Points : 44 675
    Points
    44 675
    Par défaut
    par-contre la méthode 2 n'est pas compatible avec IE
    Je parlais bien de la méthode 2.

Discussions similaires

  1. Réponses: 8
    Dernier message: 29/03/2018, 17h49
  2. onchange sur un <select> ne fonctionne pas
    Par steph68b dans le forum Général JavaScript
    Réponses: 10
    Dernier message: 16/08/2013, 13h56
  3. [XL-2010] Méthode select ne fonctionne pas sur 2010 contrairement à 2003
    Par will-1981 dans le forum Excel
    Réponses: 1
    Dernier message: 15/03/2013, 16h22
  4. Réponses: 7
    Dernier message: 10/01/2006, 10h27
  5. Un Hint sur un PopupMenu ne fonctionne pas !!??
    Par momox dans le forum C++Builder
    Réponses: 6
    Dernier message: 26/05/2003, 16h48

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo