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

Updated navbar.js #3145

Merged
merged 1 commit into from
Nov 9, 2024
Merged

Updated navbar.js #3145

merged 1 commit into from
Nov 9, 2024

Conversation

harshshukla06
Copy link
Contributor

Added the debounce function and then wrapped the scroll event handler with it.

When applied to scroll events, debouncing ensures the event handler is only executed after the user has stopped scrolling for a specified delay (e.g., 20 milliseconds). This prevents the event from firing continuously, which can be costly in terms of performance, especially when the page has visual updates during scrolling.

added the debounce function and then wrapped the scroll event handler with it.
Copy link

vercel bot commented Nov 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fin-veda ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 9, 2024 4:14pm

Copy link

github-actions bot commented Nov 9, 2024

Thank you for submitting your chaotic pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our chaotic CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Copy link

netlify bot commented Nov 9, 2024

Deploy Preview for getfinveda ready!

Name Link
🔨 Latest commit c8894bb
🔍 Latest deploy log https://app.netlify.com/sites/getfinveda/deploys/672f8a3b672e41000860fdd4
😎 Deploy Preview https://deploy-preview-3145--getfinveda.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 40
Accessibility: 63
Best Practices: 75
SEO: 73
PWA: 50
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🎉 Thank you for your contribution! Your pull request has been submitted successfully. A maintainer from FinVeda team will review it as soon as possible. We appreciate your support in making this project better

@deepeshmlgupta deepeshmlgupta merged commit 55a63f8 into ayush-that:main Nov 9, 2024
8 checks passed
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.

2 participants