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.

Sign up for GitHub

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

Jump to bottom

Fix prefixed animation name replacement #1556

Merged
merged 1 commit into from
Jun 22, 2018
Merged

Fix prefixed animation name replacement #1556

merged 1 commit into from
Jun 22, 2018

Conversation

kaisermann
Copy link
Member

@kaisermann kaisermann commented Jun 22, 2018 •
edited
Loading

I forgot to handle the renaming of animation and animation-name in #1505

(I also stopped editorconfig from inserting newlines at the end of expected.css files)

@kaisermann kaisermann force-pushed the fix-prefixed-animation-property branch from 478bb7a to d00e87d Compare June 22, 2018 18:00
@kaisermann kaisermann force-pushed the fix-prefixed-animation-property branch from d00e87d to fe9a68d Compare June 22, 2018 18:01
@Rich-Harris Rich-Harris merged commit 3560b86 into sveltejs:master Jun 22, 2018
@Rich-Harris
Copy link
Member

thank you! will cut a release shortly

@kaisermann kaisermann deleted the fix-prefixed-animation-property branch June 25, 2018 14:54
@kaisermann
Copy link
Member Author

Oh, I think I put some extras () on the remove prefix regex. It can be: ^-(webkit|moz|o|ms)-

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kaisermann @Rich-Harris