Skip to content

Commit d407390

Browse files
committed
Fix sort at-rules by their parameter name
1 parent 3eb9d8f commit d407390

File tree

4 files changed

+5
-3
lines changed

4 files changed

+5
-3
lines changed

index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ module.exports = postcss.plugin('postcss-sort', function (opts) {
100100
var atruleParameter = /^[\w-]+/.exec(node.params);
101101

102102
if (atruleParameter && atruleParameter.length) {
103-
var sortNameExtended = sortName + ' ' + atruleParameter[0];
103+
var sortNameExtended = atruleName + ' ' + atruleParameter[0];
104104

105105
if (order[sortNameExtended]) {
106106
sortName = sortNameExtended;

test/fixtures/at-rules-by-parameter.css

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
.block {
2+
@some-rule hello;
23
border: none;
34
@mixin clearfix;
45
@mixin island;
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
.block {
22
@mixin island;
33
border: none;
4+
@some-rule hello;
45
@mixin clearfix;
56
}

test/test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -109,9 +109,9 @@ test('Should sort complex case with leftovers', t => {
109109
] });
110110
});
111111

112-
test('Should sort at-rules by they parameter name', t => {
112+
test('Should sort at-rules by their parameter name', t => {
113113
return run(t, 'at-rules-by-parameter', { 'sort-order': [
114-
['@mixin', 'border', '@mixin clearfix']
114+
['@mixin', 'border', '@some-rule hello', '@mixin clearfix']
115115
] });
116116
});
117117

0 commit comments

Comments
 (0)