Merge pull request #40 from tbela99/expand_nesting_rules

incorrectly expand nesting rules #38

Author: tbela99

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## V0.5.4
+
+- [x] incorrectly expand css nesting rules
+
 ## V0.5.3
 
 - [x] incorrectly expand css nesting rules

dist/index-umd-web.js

@@ -7201,11 +7201,17 @@
                     const rule = ast.chi[i];
                     if (!rule.sel.includes('&')) {
                         const selRule = splitRule(rule.sel);
-                        selRule.forEach(arr => combinators.includes(arr[0].charAt(0)) ? arr.unshift(ast.sel) : arr.unshift(ast.sel, ' '));
-                        rule.sel = selRule.reduce((acc, curr) => {
-                            acc.push(curr.join(''));
-                            return acc;
-                        }, []).join(',');
+                        if (selRule.length > 1) {
+                            const r = ':is(' + selRule.map(a => a.join('')).join(',') + ')';
+                            rule.sel = splitRule(ast.sel).reduce((a, b) => a.concat([b.join('') + r]), []).join(',');
+                        }
+                        else {
+                            selRule.forEach(arr => combinators.includes(arr[0].charAt(0)) ? arr.unshift(ast.sel) : arr.unshift(ast.sel, ' '));
+                            rule.sel = selRule.reduce((acc, curr) => {
+                                acc.push(curr.join(''));
+                                return acc;
+                            }, []).join(',');
+                        }
                     }
                     else {
                         rule.sel = replaceCompound(rule.sel, ast.sel);

dist/index.cjs

@@ -7199,11 +7199,17 @@ function expandRule(node) {
                 const rule = ast.chi[i];
                 if (!rule.sel.includes('&')) {
                     const selRule = splitRule(rule.sel);
-                    selRule.forEach(arr => combinators.includes(arr[0].charAt(0)) ? arr.unshift(ast.sel) : arr.unshift(ast.sel, ' '));
-                    rule.sel = selRule.reduce((acc, curr) => {
-                        acc.push(curr.join(''));
-                        return acc;
-                    }, []).join(',');
+                    if (selRule.length > 1) {
+                        const r = ':is(' + selRule.map(a => a.join('')).join(',') + ')';
+                        rule.sel = splitRule(ast.sel).reduce((a, b) => a.concat([b.join('') + r]), []).join(',');
+                    }
+                    else {
+                        selRule.forEach(arr => combinators.includes(arr[0].charAt(0)) ? arr.unshift(ast.sel) : arr.unshift(ast.sel, ' '));
+                        rule.sel = selRule.reduce((acc, curr) => {
+                            acc.push(curr.join(''));
+                            return acc;
+                        }, []).join(',');
+                    }
                 }
                 else {
                     rule.sel = replaceCompound(rule.sel, ast.sel);

dist/lib/ast/expand.js

@@ -58,11 +58,17 @@ function expandRule(node) {
                 const rule = ast.chi[i];
                 if (!rule.sel.includes('&')) {
                     const selRule = splitRule(rule.sel);
-                    selRule.forEach(arr => combinators.includes(arr[0].charAt(0)) ? arr.unshift(ast.sel) : arr.unshift(ast.sel, ' '));
-                    rule.sel = selRule.reduce((acc, curr) => {
-                        acc.push(curr.join(''));
-                        return acc;
-                    }, []).join(',');
+                    if (selRule.length > 1) {
+                        const r = ':is(' + selRule.map(a => a.join('')).join(',') + ')';
+                        rule.sel = splitRule(ast.sel).reduce((a, b) => a.concat([b.join('') + r]), []).join(',');
+                    }
+                    else {
+                        selRule.forEach(arr => combinators.includes(arr[0].charAt(0)) ? arr.unshift(ast.sel) : arr.unshift(ast.sel, ' '));
+                        rule.sel = selRule.reduce((acc, curr) => {
+                            acc.push(curr.join(''));
+                            return acc;
+                        }, []).join(',');
+                    }
                 }
                 else {
                     rule.sel = replaceCompound(rule.sel, ast.sel);

package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tbela99/css-parser",
   "description": "CSS parser for node and the browser",
-  "version": "0.5.3",
+  "version": "0.5.4",
   "exports": {
     ".": "./dist/node/index.js",
     "./umd": "./dist/index-umd-web.js",

src/lib/ast/expand.ts

@@ -82,14 +82,25 @@ function expandRule(node: AstRule): Array<AstRule | AstAtRule> {
                 if (!rule.sel.includes('&')) {
 
                     const selRule: string[][] = splitRule(rule.sel);
-                    selRule.forEach(arr => combinators.includes(arr[0].charAt(0)) ? arr.unshift(ast.sel) : arr.unshift(ast.sel, ' '));
 
-                    rule.sel = selRule.reduce((acc: string[], curr: string[]) => {
+                    if (selRule.length > 1) {
 
-                        acc.push(curr.join(''));
+                        const r: string = ':is(' + selRule.map(a => a.join('')).join(',') + ')';
+
+                        rule.sel = splitRule(ast.sel).reduce((a, b) => a.concat([b.join('') + r]), <string[]>[]).join(',');
+                    }
+
+                    else {
+
+                        selRule.forEach(arr => combinators.includes(arr[0].charAt(0)) ? arr.unshift(ast.sel) : arr.unshift(ast.sel, ' '));
+                        rule.sel = selRule.reduce((acc: string[], curr: string[]) => {
+
+                            acc.push(curr.join(''));
+
+                            return acc;
+                        }, <string[]>[]).join(',');
+                    }
 
-                        return acc;
-                    }, <string[]>[]).join(',');
                 } else {
 
                     rule.sel = replaceCompound(rule.sel, ast.sel);