jonkemp/inline-css

Add sortAlphabetically option

Open

#61 aberto em 30 de mar. de 2017

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

Métricas do repositório

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

Description

I would appreciate adding of sortAlphabetically option, which would be set to true in default (to be compatible with older versions). That's because I need special CSS attributes for older IEs starting with underscore to be defined after their standard relatives, but inline-css sorts them before their standard relatives. That means the standard attribute overwrites its behaviour.

Before inlining:

div {
    width: auto;
    _width: 230px;
}

After inlining:

<div style="_width:230px;width:auto;">...</div>

which means that even in old IEs 230px is overwritten by auto.

Guia do colaborador

Direção de pesquisa
Implemente uma nova opção 'sortAlphabetically' com valor padrão true, garantindo que as propriedades CSS sejam ordenadas alfabeticamente durante a injeção inline, mantendo a compatibilidade retroativa ao permitir que os usuários a definam como false quando precisarem de ordenação personalizada, como hacks para IE.
Pilha de tecnologia
javascriptnodejs
Domain
backend
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.
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
GitNode.js
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75