facebookarchive/draft-js

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

Open

#1 116 ouverte le 3 avr. 2017

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)JavaScript (22 554 stars) (2 646 forks)batch import
enhancementgood first issue

Description

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.

Guide contributeur

Stack technique
javascriptreact
Domaine
frontend
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
ReactJavaScriptfamiliarity with draft js codebase
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Review the changes in PR #1108 that added boolean params to getCharacterRemovalRange.js. Then refactor the function to split into two methods (e.g., for forward and backward deletion) and update all callsites. Ensure existing tests pass and add tests if needed. See issue comments for discussion.