emotion-js/facepaint

Adding nested selectors at a given breakpoint

Open

#12 aberto em 12 de abr. de 2018

Ver no GitHub
 (0 comments) (2 reactions) (0 assignees)JavaScript (12 forks)github user discovery
feature requesthelp wanted

Métricas do repositório

Stars
 (594 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

I have pseudo-selector classes that only appear at my larger breakpoint. Currently it appears that I have to put every property within them inside an array:

mq({
  '&>li:first-child': {
    background: [undefined, 'green'],
    width: [undefined, 200],
  },
})

(In practice there's many more properties in there.)

Is there any neater way I could write this, and if not could one be added? I feel like I should be able to do either:

mq({
  '&>li:first-child': [undefined,
    background: 'green',
    width: 200,
  ],
})

or

mq([undefined, {
  '&>li:first-child': {
    background: 'green',
    width: 200,
  },
}])

Guia do colaborador

Direção de pesquisa
Estude a implementação atual de como os arrays de breakpoint são processados na biblioteca. Procure a lógica de parsing que lida com valores de array e veja como estendê la para suportar a sintaxe de objeto aninhado para seletores em breakpoints específicos.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Funcionalidade
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.
Meio dia
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
JavaScriptCSS
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75