fix: don't generate invalid escaping classes (#9)

css-modules · Mar 4, 2019 · d6e38e6 · d6e38e6
1 parent 67697e6
commit d6e38e6
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 4 deletions.
diff --git a/src/index.js b/src/index.js
@@ -58,9 +58,9 @@ const processor = postcss.plugin('postcss-modules-scope', function(options) {
 
     const exports = Object.create(null);
 
-    function exportScopedName(name) {
+    function exportScopedName(name, rawName) {
       const scopedName = generateScopedName(
-        name,
+        rawName ? rawName : name,
         css.source.input.from,
         css.source.input.css
       );
@@ -78,11 +78,17 @@ const processor = postcss.plugin('postcss-modules-scope', function(options) {
           return node;
         case 'class':
           return selectorParser.className({
-            value: exportScopedName(node.value),
+            value: exportScopedName(
+              node.value,
+              node.raws && node.raws.value ? node.raws.value : null
+            ),
           });
         case 'id': {
           return selectorParser.id({
-            value: exportScopedName(node.value),
+            value: exportScopedName(
+              node.value,
+              node.raws && node.raws.value ? node.raws.value : null
+            ),
           });
         }
       }

diff --git a/test/test-cases/export-class/expected.css b/test/test-cases/export-class/expected.css
@@ -6,6 +6,10 @@
   color: red;
 }
 
+._input__\:\:\:\: {
+  color: rebeccapurple;
+}
+
 @media (max-width: 520px) {
   /* selector doubled to increase specificity */
   ._input__exportName._input__exportName {
@@ -19,5 +23,6 @@
 
 :export {
   exportName: _input__exportName;
+  ::::: _input__\:\:\:\:;
   newExport: _input__newExport;
 }
diff --git a/test/test-cases/export-class/source.css b/test/test-cases/export-class/source.css
@@ -6,6 +6,10 @@
   color: red;
 }
 
+:local(.\:\:\:\:) {
+  color: rebeccapurple;
+}
+
 @media (max-width: 520px) {
   /* selector doubled to increase specificity */
   :local(.exportName):local(.exportName) {