gajus/babel-plugin-react-css-modules

Classnames have trailing whitespace when spread is used

Open

#262 ouverte le 30 juin 2019

Voir sur GitHub
 (3 commentaires) (4 réactions) (0 assignés)JavaScript (176 forks)batch import
enhancementhelp wanted

Métriques du dépôt

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

Description

Example: JSX:

<div styleName="classA classB" {...someProps} />

Output:

<div className="classA classB " />

No trailing whitespace is added when spread is not used.

Seems like a small issue, but it breaks snapshots or className checks in tests when upgrading.

Guide contributeur

Direction de recherche
Étudiez le code source de babel plugin react css modules, en particulier la partie qui gère les attributs de propagation JSX. Examinez comment styleName est converti en className et identifiez où l'espace blanc de fin est ajouté. Le problème se situe probablement dans la logique de concaténation de chaînes. Vérifiez les fichiers de test pour les tests existants et envisagez d'ajouter un cas de test pour les attributs de propagation.
Stack technique
javascriptreact
Domaine
frontend
Type d'issue
Bug
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptReact
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70