[css-minifier] Correctly parsed and compressed <keyframes-name> = <string>

[yisibl]

Spec

<keyframes-name> = <custom-ident> | <string>

https://drafts.csswg.org/css-animations/#typedef-keyframes-name

Often <string> can be compressed to <custom-ident>, However, CSS-wide keywords and none must remain in quotation marks.

Input

@keyframes "one two" {
  0% {
    left: 100px;
  }
}

@keyframes "none" {
  to {
    right: 100px;
  }
}

.foo {
  animation-name: "none", "unset", "one two";
}

Actual output

This WPT PR has more detailed test cases. web-platform-tests/wpt#35903

@keyframes ""{0%{left:100px}}@keyframes ""{to{right:100px}}.foo{animation-name:"none","unset","one two"}

Expected

@keyframes one\ two{0%{left:100px}}@keyframes"none"{to{right:100px}}.foo{animation-name:"none","unset",one\ two}

[yisibl]

Thanks for the quick fix, but this issue is also expected to compress <string> and handle CSS-wide keywords correctly.

The ultimate goal is for esbuild to parse <keyframes-name> in its entirety.

I think @CGQAQ's PR #2556 is working on these issues and hopefully will reopen this issue.