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 (2 646 forks)batch import
enhancementgood first issue

Métriques du dépôt

Stars
 (22 554 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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

Direction de recherche
Identifiez les paramètres booléens dans getCharacterRemovalRange.js, créez deux fonctions distinctes sans indicateurs booléens et mettez à jour tous les sites d'appel pour utiliser la fonction appropriée.
Stack technique
javascriptreact
Domaine
frontend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80