Ink-text-input: gérerSoumettre

Créé le 7 mars 2019 · 11Commentaires · Source: vadimdemedes/ink-text-input

Le démarrage rapide sur readme.md mentionnait this.handleSubmit = this.handleSubmit.bind(this); , qui est vide et déclenchera une erreur.

De plus, onChange est géré correctement alors que onSubmit ne l'est pas.

Peut-être devons-nous gérer ENTER ?

if(s === ENTER && this.props.onSubmit && value !== originalValue){
  return this.props.onSubmit(value);
}

Source

entrptaher

👍1

Commentaire le plus utile

3rded - pour l'instant j'ai copié le code dans un fichier localement et ajouté ma propre méthode onSubmit

danrspencer le 8 mars 2019

😄2 👍1

Tous les 11 commentaires

J'étais justement sur le point d'ouvrir un sujet pour ça. Ce serait génial ! (Et je vois que le composant d'entrée de mot de passe obsolète a onSubmit )

SimenB le 8 mars 2019

3rded - pour l'instant j'ai copié le code dans un fichier localement et ajouté ma propre méthode onSubmit

danrspencer le 8 mars 2019

😄2 👍1

4thed - J'ai également ajouté mon propre onSubmit personnalisé.

jcar787 le 8 mars 2019

@danrspencer @jcar787 faisons une pull request, qu'en dites-vous ? :RÉ

entrptaher le 9 mars 2019

👍1

@entrptaher Je pense que vous devriez le faire puisque vous avez découvert le problème en premier et ajouté l'essentiel. 😊
Merci pour l'essentiel cependant.

jcar787 le 9 mars 2019

Vous avez raison.
Cependant, j'ai creusé profondément et j'ai trouvé que j'avais un problème avec le curseurOffset lorsque je soumets en utilisant ENTER.
Alors j'ai posé la question :D

entrptaher le 9 mars 2019

Version crochet . Cependant, vous n'en tirez pas la valeur.

mAAdhaTTah le 11 mars 2019

👍1

Je suppose que je l'ai accidentellement supprimé lors de la migration vers Ink 2 ou une version antérieure 🤷‍♂️ Si quelqu'un est prêt à aider, ce serait formidable ! Sinon, je m'en occuperai un peu plus tard cette semaine.