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

add once option to amp-position-observer #11047

Closed
aghassemi opened this issue Aug 23, 2017 · 3 comments
Closed

add once option to amp-position-observer #11047

aghassemi opened this issue Aug 23, 2017 · 3 comments

Comments

@aghassemi
Copy link
Contributor

There are cases where we want amp-position-observer to only run once (e.g. fade in images). Support that option.

@aghassemi aghassemi self-assigned this Aug 23, 2017
@ampprojectbot ampprojectbot added this to the Pending Triage milestone Sep 12, 2017
@ampprojectbot
Copy link
Member

This issue seems to be in Pending Triage for awhile. @aghassemi Please triage this to an appropriate milestone.

@mustafaj17
Copy link

would really be useful, anyone found any work arounds ?

@aghassemi
Copy link
Contributor Author

aghassemi commented Jan 19, 2018

@mustafaj17 are you using it for scroll-bound animations or just triggering animations when something comes into the view? If the latter, you can work around this by not providing a first keyframe for your animation, this way the second time it plays, it would try to go from the current state (which would be the last keyframe of previous run) to last keyframe again (which would be the same) and effectively it won't do anything.

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

No branches or pull requests

4 participants