fixes bug trullock#331

akuegel · Jul 21, 2022 · 6994821 · 6994821
1 parent 7a62a6f
commit 6994821
Show file tree

Hide file tree

Showing 5 changed files with 27 additions and 1 deletion.
diff --git a/src/NUglify.Tests/Css/Bugs.cs b/src/NUglify.Tests/Css/Bugs.cs
@@ -128,5 +128,13 @@ public void Bug309()
 	--custom-property: 0px; 
 }").Code);
         }
+
+
+        [Test]
+        public void Bug331()
+        {
+	        TestHelper.Instance.RunTest();
+        }
+
     }
 }
diff --git a/src/NUglify.Tests/NUglify.Tests.csproj b/src/NUglify.Tests/NUglify.Tests.csproj
@@ -401,6 +401,9 @@
     <Content Include="TestData\CSS\Expected\Bugs\Bug250.css">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </Content>
+    <Content Include="TestData\CSS\Expected\Bugs\Bug331.css">
+      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+    </Content>
     <Content Include="TestData\CSS\Expected\Bugs\Bug74.css">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </Content>
@@ -431,6 +434,9 @@
     <Content Include="TestData\CSS\Input\AtRules\RazorEscapedDoubleAt.css">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </Content>
+    <Content Include="TestData\CSS\Input\Bugs\Bug331.css">
+      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+    </Content>
     <Content Include="TestData\CSS\Input\Bugs\Bug270.css">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </Content>

diff --git a/src/NUglify.Tests/TestData/CSS/Expected/Bugs/Bug331.css b/src/NUglify.Tests/TestData/CSS/Expected/Bugs/Bug331.css
@@ -0,0 +1 @@
+.selector{--variable:;color:#f00}.selector2{--variable:}
diff --git a/src/NUglify.Tests/TestData/CSS/Input/Bugs/Bug331.css b/src/NUglify.Tests/TestData/CSS/Input/Bugs/Bug331.css
@@ -0,0 +1,7 @@
+.selector {
+	--variable: ;
+	color: red;
+}
+.selector2 {
+	--variable:;
+}
diff --git a/src/NUglify/Css/CssParser.cs b/src/NUglify/Css/CssParser.cs
@@ -2627,7 +2627,11 @@ Parsed ParseDeclaration()
                 }
                 else 
                 {
-                    m_parsingColorValue = MightContainColorNames(propertyName);
+                    // valueless variable declarations are legal
+	                if (propertyName.StartsWith("--") && CurrentTokenText == ";")
+		                return Parsed.True;
+
+	                m_parsingColorValue = MightContainColorNames(propertyName);
                     parsed = ParseExpr();
                     m_parsingColorValue = false;
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		.selector{--variable:;color:#f00}.selector2{--variable:}