jonkemp/inline-css

Add sortAlphabetically option

Open

#61 opened on Mar 30, 2017

View on GitHubContributor guide
 (4 comments) (0 reactions) (0 assignees)JavaScript (79 forks)github user discovery
feature requesthelp wanted

Repository metrics

Stars
 (438 stars)
PR merge metrics
 (No merged PRs in 30d)

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.

Contributor guide

Research direction
Implement a new 'sortAlphabetically' option that defaults to true, ensuring CSS properties are sorted alphabetically when inlined, while maintaining backward compatibility by allowing users to set it to false when they need custom ordering like IE hacks.
Tech stack
javascriptnodejs
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitNode.js
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75