gajus/babel-plugin-react-css-modules

Classnames have trailing whitespace when spread is used

Open

#262 aberto em 30 de jun. de 2019

Ver no GitHub
 (3 comments) (4 reactions) (0 assignees)JavaScript (176 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (2.044 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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.

Guia do colaborador

Direção de pesquisa
Investigue o código fonte do babel plugin react css modules, especialmente a parte que lida com atributos de propagação JSX. Veja como styleName é convertido para className e identifique onde o espaço em branco à direita é adicionado. O problema provavelmente está na lógica de concatenação de strings. Verifique os arquivos de teste para testes existentes e considere adicionar um caso de teste para atributos de propagação.
Pilha de tecnologia
javascriptreact
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptReact
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70