react / react/react

Form input name='nodeName' breaks onSubmit event handling

Ouverte
#6,284 9 commentaires 2 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Component: DOM Type: Bug
Langage dominant
JavaScript
Étoiles
251k
Forks
51.4k
Merge moyen
2 j 4 h
PR mergées (30 j)
53

Description

It happened that I stumbled on following edge case. If you add name='nodeName' attribute to form's input, at some point of React event handling (ChangeEventPlugin.js: shouldUseChangeEvent()) it will call: elem.nodeName && elem.nodeName.toLowerCase(), but coincidentally nodeName property refers to input and invocation fails.

Here's a jsFiddle example

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par la reproduction jsFiddle et inspectez ChangeEventPlugin.js, en particulier shouldUseChangeEvent(), où l’accès à nodeName de l’input est effectué. Confirmez l’échec lorsqu’un input de formulaire est nommé nodeName, puis vérifiez que onSubmit et la gestion d’événements associée fonctionnent sans cette collision.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript, react
Domaine
frontend
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.