facebookarchive/draft-js

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

Aperta

#991 aperta il 31 gen 2017

Vedi su GitHubGuida contributor
 (4 commenti) (1 reazione) (0 assegnatari)JavaScript (2646 fork)batch import
bughelp wanted

Metriche repository

Star
 (22.554 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina il codice di gestione dell'incolla di Draft.js per capire perché la selezione non viene rimossa prima del mouseup. Concentrati sul metodo `handlePastedText` e sull'interazione con gli eventi di selezione del browser.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75