Skip to content

Conversation

@Parwathi-Jayaram
Copy link

Description

I have increased the auto scroll timing. Now the timer resets to zero when the users scroll to next part so the users would have a chance to read the content properly.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

Related Issue

Closes #44

Changes Made

Please list the specific changes made in this PR:

  1. I have the increased the auto scroll timing.
  2. I have reset the timing to zero whenever users click next.
  3. Now the auto scroll won't interfere with the users.

Testing Done

Please describe how you tested your changes:

  • Tested in development mode (npm run dev)
  • Tested on multiple browsers (microsoft edge, google chrome)
  • Checked for console errors

Screenshots (if applicable)

Recording.2026-01-11.140158.mp4

Checklist

IMPORTANT: Please ensure all items are checked before submitting. PRs that don't follow these guidelines may be closed.

  • I have read and followed all guidelines in CONTRIBUTING.md
  • I have used the correct branch naming convention (feature/, fix/, docs/, etc.)
  • My commits follow Conventional Commits format (feat:, fix:, docs:, etc.)
  • I have performed a self-review and tested my changes thoroughly
  • CRITICAL: I confirm ONLY my intended changes are included (no unrelated changes, debug code, or breaking changes)
  • I have updated relevant documentation if needed
  • I have tested the production build locally
  • My changes are responsive and work on mobile devices

Additional Notes

Any additional information that would be helpful for reviewers?


Remember: Quality over quantity. A well-tested, properly documented small PR is better than a large, rushed contribution.

For more details, please refer to our Contributing Guidelines.

@vercel
Copy link

vercel bot commented Jan 11, 2026

@Parwathi-Jayaram is attempting to deploy a commit to the Akash Kumar's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Jan 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
web Ready Ready Preview, Comment Jan 13, 2026 7:27pm

@akash-kumar-dev
Copy link
Member

Hi @Parwathi-Jayaram, could you please address this lint warning?

image

@Parwathi-Jayaram
Copy link
Author

Screenshot 2026-01-14 002043 The issue is resolved now

@Parwathi-Jayaram
Copy link
Author

@akash-kumar-dev pls check the fix

@akash-kumar-dev
Copy link
Member

akash-kumar-dev commented Jan 20, 2026

Hi @Parwathi-Jayaram,
Thanks for your contribution, the changes look good! Due to the force push, some already merged files and code are also in the code review. While this isn't a major issue, it would be great if you could open a new pull request from a different branch without force pushing, to maintain a cleaner and more consistent commit history.
and also, try to review your changes locally before committing to ensure that only your changes are included in the review, and no other parts of the code.

@Parwathi-Jayaram
Copy link
Author

ok I will do that and close this pull request then

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.

[BUG] The auto scroll timings in home page is too fast

2 participants