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: Navigation state provider and hook #6709
feat: Navigation state provider and hook #6709
Conversation
Temporary workaround for nodejs#6409 Get more info here - nodejs#6675 Signed-off-by: abizek <abishekilango@protonmail.com>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@abizek this is looking good! I left a few minor comments/suggestions and ideas :) |
Signed-off-by: abizek <abishekilango@protonmail.com>
@ovflowd I refactored the effect and responded to some of your comments. Please take a look. |
Lighthouse Results
|
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.
As much as I'd like to nitpick more, I believe we should land this as it is.
Awesome work, @abizek; Thank you so much for going back and forth and helping us out here, I tenfold appreciate your work here!
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.
clean first contribution !
Signed-off-by: abizek <abishekilango@protonmail.com>
Description
Adds a Navigation State provider and hook to track scroll position and restore it on remount.
Validation
Related Issues
Temporary workaround for #6409
Get more info here - #6675 (comment)
Check List
npm run format
to ensure the code follows the style guide.npm run test
to check if all tests are passing.npx turbo build
to check if the website builds without errors.