fix #1665: don’t remove empty @Keyframes (#1669)

evanw · Oct 12, 2021 · 150882d · 150882d
1 parent c1a3b8c
commit 150882d
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/internal/css_parser/css_parser.go b/internal/css_parser/css_parser.go
@@ -279,11 +279,6 @@ skipRule:
 		rule := rules[i]
 
 		switch r := rule.Data.(type) {
-		case *css_ast.RAtKeyframes:
-			if len(r.Blocks) == 0 {
-				continue
-			}
-
 		case *css_ast.RKnownAt:
 			if len(r.Rules) == 0 {
 				continue

diff --git a/internal/css_parser/css_parser_test.go b/internal/css_parser/css_parser_test.go
@@ -899,7 +899,7 @@ func TestEmptyRule(t *testing.T) {
 	expectPrintedMangle(t, "div {}", "")
 	expectPrintedMangle(t, "@media screen {}", "")
 	expectPrintedMangle(t, "@page { @top-left {} }", "")
-	expectPrintedMangle(t, "@keyframes test { from {} to {} }", "")
+	expectPrintedMangle(t, "@keyframes test { from {} to {} }", "@keyframes test {\n}\n")
 
 	expectPrinted(t, "$invalid {}", "$invalid {\n}\n")
 	expectPrinted(t, "@page { color: red; @top-left {} }", "@page {\n  color: red;\n  @top-left {\n  }\n}\n")