gajus/babel-plugin-react-css-modules

Classnames have trailing whitespace when spread is used

Offen

#262 geöffnet am 30.06.2019

Auf GitHub ansehen
 (3 Kommentare) (4 Reaktionen) (0 zugewiesene Personen)JavaScript (159 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (2.034 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche den Quellcode von babel plugin react css modules, insbesondere den Teil, der JSX Spread Attribute verarbeitet. Sieh dir an, wie styleName in className umgewandelt wird, und identifiziere, wo nachgestellter Leerraum hinzugefügt wird. Das Problem liegt wahrscheinlich in der String Verkettungslogik. Überprüfe die Testdateien auf vorhandene Tests und erwäge, einen Testfall für Spread Attribute hinzuzufügen.
Tech Stack
javascriptreact
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptReact
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70