facebookarchive/draft-js

Improve style by removing boolean param in `getCharacterRemovalRange.js`

Open

#1116 aperta il 3 apr 2017

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (2646 fork)batch import
enhancementgood first issue

Metriche repository

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

Descrizione

A recent change added two boolean flag params to getCharacterRemovalRange.js. Boolean params generally make code less readable and maintainable - this is not worth blocking a PR at this time, but it would be great to update this file and the callsites to call two methods instead of passing in different boolean params.

Guida contributor

Direzione di ricerca
Rivedi le modifiche nella PR #1108 che hanno aggiunto parametri booleani a getCharacterRemovalRange.js. Quindi rifattorizza la funzione per dividerla in due metodi (ad esempio, per cancellazione in avanti e indietro) e aggiorna tutti i punti di chiamata. Assicurati che i test esistenti passino e aggiungi test se necessario. Vedi i commenti dell'issue per la discussione.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
ReactJavaScriptfamiliarity with draft js codebase
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60