andreypopp/react-css-components

Allow className prop to apply classes to components

Open

#13 aberto em 13 de jul. de 2016

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)JavaScript (676 stars) (23 forks)batch import
enhancementhelp wanted

Description

At the moment, if you import a component from your CSS, using the className prop to pass global classes in doesn't seem to work.

Is there a way to make these carry through to the underlying markup to allow things like global layout classes?

Guia do colaborador

Pilha de tecnologia
reactcssjavascript
Domain
frontend
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
mostly clear
Prerequisites
ReactCSS ModulesProp forwarding
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75
Direção de pesquisa
The issue requests that the `className` prop be forwarded to the underlying DOM elements. Investigate the component implementation in `src/`, likely using CSS Modules, to see how classes are currently applied. Ensure that the passed `className` is merged with the module's generated class. Review existing tests and update them to cover this case. Check for any breaking changes or edge cases with multiple class names.