jonkemp/inline-css

Add sortAlphabetically option

Open

#61 ouverte le 30 mars 2017

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)JavaScript (80 forks)github user discovery
feature requesthelp wanted

Métriques du dépôt

Stars
 (440 stars)
Métriques de merge PR
 (Métriques PR en attente)

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.

Guide contributeur

Direction de recherche
Implémentez une nouvelle option 'sortAlphabetically' avec une valeur par défaut true, garantissant que les propriétés CSS sont triées alphabétiquement lors de l'intégration, tout en maintenant la rétrocompatibilité en permettant aux utilisateurs de la définir sur false lorsqu'ils ont besoin d'un ordre personnalisé comme les hacks IE.
Stack technique
javascriptnodejs
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitNode.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75