Skip to content

Revert "⏪ Temporarily revert lazy polyfill web animations"#29775

Merged
kristoferbaxter merged 3 commits intomasterfrom
revert-29772-revert-polyfill-animations
Aug 11, 2020
Merged

Revert "⏪ Temporarily revert lazy polyfill web animations"#29775
kristoferbaxter merged 3 commits intomasterfrom
revert-29772-revert-polyfill-animations

Conversation

@kristoferbaxter
Copy link
Copy Markdown
Contributor

Reverts #29772

@google-cla google-cla bot added the cla: yes label Aug 10, 2020
@kristoferbaxter
Copy link
Copy Markdown
Contributor Author

Restarting Dist Tests job, it appears to have failed for an unrelated reason.

@kristoferbaxter kristoferbaxter marked this pull request as ready for review August 11, 2020 17:28
@amp-owners-bot amp-owners-bot bot requested a review from estherkim August 11, 2020 17:28
@kristoferbaxter kristoferbaxter requested a review from rsimha August 11, 2020 17:29
@kristoferbaxter
Copy link
Copy Markdown
Contributor Author

Thanks @jridgewell who figured out how to correctly install the polyfill into the AMP window within a module.

@kristoferbaxter kristoferbaxter merged commit 0e06870 into master Aug 11, 2020
@rsimha rsimha deleted the revert-29772-revert-polyfill-animations branch October 19, 2020 22:35
ed-bird pushed a commit to ed-bird/amphtml that referenced this pull request Dec 10, 2020
…t#29775)

* Revert "⏪ Temporarily revert lazy polyfill web animations (ampproject#29772)"

This reverts commit 933c001.

* Install polyfill into child window

* Update install-polyfill.js

Cleanup

Co-authored-by: Justin Ridgewell <jridgewell@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants