Skip to content

Conversation

@guyromellemagayano
Copy link
Contributor

@guyromellemagayano guyromellemagayano commented Nov 15, 2025

This pull request enhances Animate On Scroll (AOS) initialization, improves smooth scroll integration, and adjusts animation delays for better user experience and SSR compatibility.

Key changes:

  • Adds client-side checks in aos.js for SSR compatibility
  • Configures AOS with proper settings to ensure smooth animations
  • Streamlines vue-smooth-scroll.js integration and includes documentation
  • Optimizes onScrollToTop in MainLayout.vue for cleaner scrolling logic
  • Adjusts AOS delays in IndexPage.vue for staggered animation timing

These changes refine animation behavior, improve compatibility, and enhance overall user experience.

@guyromellemagayano guyromellemagayano changed the title refactor(aos.js, vue-smooth-scroll.js, IndexPage.vue): Enhance AOS initialization and adjust delays Optimize AOS setup with client-side checks and improve animation timing Nov 16, 2025
@jofftiquez
Copy link
Member

This is approved, however, kindly provide a proof of testing. A loom view would suffice. Thanks.

…nitialization and remove delays

Refactor AOS initialization in `aos.js` to remove client-side checks and streamline configuration. Update `IndexPage.vue` to eliminate AOS delay attributes for a more uniform animation effect.

Changes:
- Simplified AOS initialization in `aos.js`
- Removed delay attributes from AOS elements in `IndexPage.vue` for consistent animation timing
@guyromellemagayano
Copy link
Contributor Author

@jofftiquez I prepared a Loom recording demonstrating the PR changes here:

https://www.loom.com/share/55874134b10f489a9dc5612a48d14d6e

Copy link
Member

@jofftiquez jofftiquez left a comment

Choose a reason for hiding this comment

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

LGTM thanks

@jofftiquez jofftiquez merged commit fe10b2c into OSSPhilippines:main Nov 23, 2025
@guyromellemagayano guyromellemagayano deleted the refactor/aos-adjustments branch November 25, 2025 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants