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

keyframes better support older iOS (8.x) webkit #720

Merged
merged 2 commits into from Apr 20, 2017

Conversation

tyfoo
Copy link
Contributor

@tyfoo tyfoo commented Apr 20, 2017

When using styled-components in one of our projects we noticed that on older iOS devices (8.x, iPhone 6 and 6 Plus) animations didn't work.

After some investigation we've found http://stackoverflow.com/questions/9211261/css3-animation-not-working-in-safari and after some experimentation, the solution suggested in http://stackoverflow.com/a/34013139 worked.

This PR adds changes keyframes to generate both @keyframes {name} and @-webkit-keyframes {name}.

@mxstbr-bot
Copy link

mxstbr-bot commented Apr 20, 2017

Warnings
⚠️ There are library changes, but not tests. That's OK as long as you're refactoring existing code

Generated by 🚫 dangerJS

@kitten
Copy link
Member

kitten commented Apr 20, 2017

LGTM 🚢 Please note, that this is is already working in v2, since we've switched to stylis, which does vendor @Keyframes for us

@kitten kitten merged commit 60ef73e into styled-components:master Apr 20, 2017
@jesnil01
Copy link

👍 Thanks for this!

This is not only present on IOS devices but older webkit browsers on desktop as well.

Looking forward for this in a future version!

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.

None yet

4 participants