facebookarchive/draft-js

onEscape is deprecated but keyBindingFn does not catch Escape key

Open

#1839 aperta il 17 ago 2018

Vedi su GitHub
 (3 commenti) (0 reazioni) (1 assegnatario)JavaScript (2646 fork)batch import
documentationhelp wanted

Metriche repository

Star
 (22.554 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Indaga la gestione del tasto Escape in Draft.js. Esamina l'implementazione di keyBindingFn in `src/component/handlers/edit/keyBindingFn.js` e confrontala con la proprietà deprecata onEscape. Determina perché keyBindingFn non intercetta il tasto Escape e propone una soluzione. Nota che il repository è archiviato, quindi i contributi potrebbero non essere uniti.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Basic understanding of ReactFamiliarity with Draft.js editor architecture
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30