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

DS-837 animate on scroll #2542

Merged
merged 9 commits into from
Sep 26, 2022

Conversation

MarcinMr
Copy link
Collaborator

@MarcinMr MarcinMr commented Sep 15, 2022

Jira

https://pegadigitalit.atlassian.net/browse/DS-837

Summary

JS triggers animations on elements when they come within the viewport.

Details

  • After adding specific classes to wanted elements, animation triggers when elements are visible in the viewport.
  • When scrolling and elements are visible in the viewport they should be animated.

How to test

Pull the branch. Go to the Test Folder with examples of animations. Confirm that elements appear when visible in the viewport and when scrolling. Check the cascade and no-cascade behavior.

Release notes

The animation toolset is updated. Elements appear not only on the load event but they are observed. If they are on the viewport area the animation triggers.

@github-actions github-actions bot added the type: feature List this PR in the 'Features' section of the release notes. label Sep 15, 2022
…o feature/DS-837-Animate-On-Scroll-TEST-DEMO
Copy link
Collaborator

@danielamorse danielamorse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MarcinMr looks and works great.

FYI @andrewmriv and @remydenton (re: layout builder), animations will now start when an element is in the viewport rather than on load. If an element is in the viewport on load, it will animate immediately.

@danielamorse danielamorse merged commit d2593b4 into master Sep 26, 2022
@danielamorse danielamorse deleted the feature/DS-837-Animate-On-Scroll-TEST-DEMO branch September 26, 2022 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature List this PR in the 'Features' section of the release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants