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

🐛 Fix Animation.animate breakage caused by missing import. #30264

Merged
merged 7 commits into from Sep 17, 2020

Conversation

xiexr151e
Copy link
Contributor

@xiexr151e xiexr151e commented Sep 16, 2020

In fixed-layer.js, Animation is supposed to be imported but it was missing.

Fixes #30253

@google-cla google-cla bot added the cla: yes label Sep 16, 2020
@xiexr151e xiexr151e changed the title 🐛 Fix breakage caused by missing import. 🐛 Fix Animation.animate breakage caused by missing import. Sep 16, 2020
@gmajoulet
Copy link
Contributor

Looks like it's fixing #30253

src/service/fixed-layer.js Outdated Show resolved Hide resolved
test/unit/test-fixed-layer.js Show resolved Hide resolved
@xiexr151e
Copy link
Contributor Author

@ampproject/build-cop This PR is currently blocked by the handshake test.

@zhouyx
Copy link
Contributor

zhouyx commented Sep 17, 2020

#30247 has been merged to fix that test

@xiexr151e xiexr151e merged commit d83351b into ampproject:master Sep 17, 2020
ampprojectbot pushed a commit that referenced this pull request Sep 18, 2020
* fix

* added lint and test for bug

* weird space

* rename test

* dupe test

* comment

Co-authored-by: Derek Tse <derektse@google.com>
(cherry picked from commit d83351b)
ed-bird pushed a commit to ed-bird/amphtml that referenced this pull request Dec 10, 2020
…ect#30264)

* fix

* added lint and test for bug

* weird space

* rename test

* dupe test

* comment

Co-authored-by: Derek Tse <derektse@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.

🚨 Error: Animation.animate is not a function
7 participants