-
Notifications
You must be signed in to change notification settings - Fork 71
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
feat: Add animation on scroll in about-us page. #66
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for cops ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
This uses scrollTrigger plugin of gsap for adding animation to about-us content. Co-authored-by: Manan Rathi <[email protected]> Signed-off-by: Somesh Ranjan <[email protected]>
@somesh202 updates on this |
do one more thing for this PR let's get rid of |
Just add gsap and remove nuxt-gsap-module from config |
5d8720b
to
d215318
Compare
d215318
to
ce9a415
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😅 force push removed the previous changes check once properly:
- update nuxt.config.ts remove nuxt-gsap-module
- update usage of $gsap everywhere
Ohh when i meant just ☠️ i meant finally do this as well and not to undo the changes.
@@ -66,14 +66,14 @@ | |||
|
|||
<script lang="ts"> | |||
import { defineComponent } from '@nuxtjs/composition-api' | |||
import { gsap } from 'gsap' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gsap isn't there in package.json
"nuxt": "^2.15.8", | ||
"nuxt-gsap-module": "^1.7.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove nuxt-gsap-module
@king-11 Is the deploy preview of the About Us page the intended animation? The scroll trigger behavior seemed a little off🤔. |
The pinning might feel a bit off but yeah it's intended as done by @somesh202 . There is one issue though on mobile view the trailer and about us last card overlap |
Description
Add animation on scroll in about-us page:
This uses scrollTrigger plugin of gsap for adding animation to about-us content.
Fixes
Fixes #[issue number] by @[issue author]
Types of changes
Checklist: