Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] : animation referencing name starts with e + number is minized to NaN #958

Closed
fa93hws opened this issue Oct 19, 2020 · 2 comments
Closed

Comments

@fa93hws
Copy link

@fa93hws fa93hws commented Oct 19, 2020

Describe the bug

.e4yw0Q {
    animation: e4yw0Q;
}

@keyframes e4yw0Q {}

leads to
.e4yw0Q{animation:NaNyw0Q}
(It should not be NaN)

To Reproduce
https://cssnano.co/playground/#eyJpbnB1dCI6Ii5lNHl3MFEge1xuICAgIGFuaW1hdGlvbjogZTR5dzBRO1xufVxuXG5Aa2V5ZnJhbWVzIGU0eXcwUSB7fVxuXG4iLCJjb25maWciOiIvLyBjc3NuYW5vIGNvbmZpZ1xue1xuICBcInByZXNldFwiIDogXCJkZWZhdWx0XCJcbn1cbiJ9

Expected behavior
It should be .e4yw0Q{animation:e4yw0Q}

@anikethsaha
Copy link
Member

@anikethsaha anikethsaha commented Oct 23, 2020

Thanks for the issue

Duplicate #909

@anikethsaha anikethsaha linked a pull request that will close this issue Oct 23, 2020
SkReD added a commit to SkReD/cssnano that referenced this issue Oct 23, 2020
SkReD added a commit to SkReD/cssnano that referenced this issue Oct 23, 2020
@ludofischer ludofischer added this to the 5.0.0 milestone Mar 9, 2021
@ludofischer
Copy link
Collaborator

@ludofischer ludofischer commented Mar 19, 2021

Fixed in 5.0.0-rc.2

ludofischer added a commit that referenced this issue Apr 26, 2021
ludofischer added a commit that referenced this issue Apr 26, 2021
ludofischer added a commit that referenced this issue Apr 28, 2021
* test(postcss-merge-longhand): check invalid border property handling

* test(postcss-convert-values): add test case for #958

Co-authored-by: Shipov Mikhail <mshipov@yandex.ru>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants