gajus/babel-plugin-react-css-modules

Classnames have trailing whitespace when spread is used

Aperta

#262 aperta il 30 giu 2019

Vedi su GitHub
 (3 commenti) (4 reazioni) (0 assegnatari)JavaScript (159 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (2034 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Analizza il codice sorgente di babel plugin react css modules, in particolare la parte che gestisce gli attributi spread JSX. Esamina come styleName viene convertito in className e identifica dove viene aggiunto lo spazio finale. Il problema è probabilmente nella logica di concatenazione delle stringhe. Controlla i file di test per i test esistenti e considera di aggiungere un caso di test per gli attributi spread.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70