facebookarchive/draft-js

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

Aperta

#1116 aperta il 3 apr 2017

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

Metriche repository

Star
 (22.554 stelle)
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
Identificare i parametri booleani in getCharacterRemovalRange.js, creare due funzioni separate senza flag booleani e aggiornare tutti i punti di chiamata per utilizzare quella appropriata.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Refactoring
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80