facebookarchive/draft-js

onEscape is deprecated but keyBindingFn does not catch Escape key

Open

#1.839 geöffnet am 17. Aug. 2018

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (1 zugewiesene Person)JavaScript (2.646 Forks)batch import
documentationhelp wanted

Repository-Metriken

Stars
 (22.554 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie, warum onEscape in Draft.js veraltet ist. Überprüfen Sie, ob die keyBindingFn API die Escape Taste behandeln sollte. Suchen Sie nach Migrationsleitfäden oder alternativen Methoden, um die Escape Taste abzufangen.
Tech Stack
javascriptreact
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptReactDraft.js basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70