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
fix(css/parser): math functions in at-rules #6140
Conversation
@@ -2328,6 +2328,9 @@ where | |||
value: vec![], | |||
}; | |||
|
|||
println!("FUCK {:?}", function); | |||
println!("FUCK {:?}", self.ctx.block_contents_grammar); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, sorry 😄
crates/swc_css_minifier/tests/fixture/compress-calc/container/output.min.css
Show resolved
Hide resolved
@kdy1 Fixed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
swc-bump:
- swc_css_parser
- swc_css_minifier
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Automated review comment generated by auto-rebase script
Description:
Fix handle math function in
@media
and@container
at-rules + better minificatio them.There are potential case to improve
width: calc(calc(calc(100px + 100px)))
, I will improve it late, because no one write code in this style + fix keepcalc
inside@supports
, beucase it can be used to checkcalc
exists or notBREAKING CHANGE:
No
Related issue (if exists):
No