facebookarchive/draft-js

onEscape is deprecated but keyBindingFn does not catch Escape key

Open

#1.839 aberto em 17 de ago. de 2018

Ver no GitHub
 (3 comments) (0 reactions) (1 assignee)JavaScript (2.646 forks)batch import
documentationhelp wanted

Métricas do repositório

Stars
 (22.554 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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

Guia do colaborador

Direção de pesquisa
Investigue por que onEscape está obsoleto no Draft.js. Verifique se a API keyBindingFn deve lidar com a tecla Escape. Procure guias de migração ou métodos alternativos para capturar a tecla Escape.
Pilha de tecnologia
javascriptreact
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptReactDraft.js basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70