facebookarchive/draft-js

Edge case - Paste doesn't remove current selection before mouseup

Open

#991 ouverte le 31 janv. 2017

Voir sur GitHub
 (4 commentaires) (1 réaction) (0 assignés)JavaScript (2 646 forks)batch import
bughelp 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? Editor does not remove selection when pasting unless mouseup has fired (it seems that a selection isn't registered until mouseup). I use a trackpad with 3 finger drag, which has a 300-ish millisecond delay after lifting. If I paste text in that window, the following behavior occurs:

draft-js

Basically, the repro steps are:

  1. Click and drag a selection
  2. Press cmd-P before mouseup

What is the expected behavior? Paste removes currently selected text (potentially not possible). Perhaps it could disallow pasting before mouseup?

Honestly, even OS X gets this wrong. Try the same steps on the location bar.

Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js? DraftJS: 0.10.0 Browser: Chrome 56 OS: Mac OS X 10.12.2

Guide contributeur