Skip to content

When rendering declarations of KeyFrames disable minification#665

Closed
DanielJoyce wants to merge 1 commit intoparcel-bundler:masterfrom
DanielJoyce:fix-css-transform-optim-288
Closed

When rendering declarations of KeyFrames disable minification#665
DanielJoyce wants to merge 1 commit intoparcel-bundler:masterfrom
DanielJoyce:fix-css-transform-optim-288

Conversation

@DanielJoyce
Copy link

so that KeyFrame transformations are not broken

  • Added Todo to TransformList::to_css as it needs to be refactored and we need more context.
  • 4 Tests are still broken

This is mostly to get the ball rolling and to see if this fixes people's use cases around keyframes.

so that KeyFrame transformations are not broken

* Added Todo to TransformList::to_css as it needs to be refactored
  and we need more context.
* 4 Tests are still broken
@DanielJoyce
Copy link
Author

Another option is to do only whitespace minification in TransformList::to_css, and don't try and change the types.

Ideally there should be a optional minification pass added, so to_css only deals with formatting.

parse -> [ minify ] -> to_css

@DanielJoyce
Copy link
Author

Another option would be to turn minify into an enum, expressing white_space only minification and white_space + transformation

@devongovett
Copy link
Member

Closed by #694

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants