facebookarchive/draft-js

onEscape is deprecated but keyBindingFn does not catch Escape key

Open

#1 839 ouverte le 17 août 2018

Voir sur GitHub
 (3 commentaires) (0 réactions) (1 assigné)JavaScript (2 646 forks)batch import
documentationhelp wanted

Métriques du dépôt

Stars
 (22 554 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Do you want to request a feature or report a bug? Bug

What is the current behavior? keyBindingFn does not catch Escape key

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. You can use this jsfiddle to get started: https://jsfiddle.net/stopachka/m6z0xn4r/. https://codesandbox.io/s/6jw8mzxl3n

What is the expected behavior? keyBindingFn should catch Escape key OR onEscape should not trigger a deprecation warning OR there should be an alternative way of catching Escape key

Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js? Draft.js 0.10.5

Guide contributeur

Direction de recherche
Enquêtez sur la raison pour laquelle onEscape est déprécié dans Draft.js. Vérifiez si l'API keyBindingFn doit gérer la touche Échap. Recherchez des guides de migration ou des méthodes alternatives pour capturer la touche Échap.
Stack technique
javascriptreact
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptReactDraft.js basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70